Skip to content

Conversation

@kaixin-hc
Copy link
Contributor

@kaixin-hc kaixin-hc commented May 1, 2024

What is the purpose of this pull request?

  • Documentation update
  • Bug fix
  • Feature addition or enhancement
  • Code maintenance
  • DevOps
  • Improve developer experience
  • Others, please explain:

Fixes #2543

Overview of changes:
Change link endings from .com to .app for the netlify deployments

Anything you'd like to highlight/discuss:

Testing instructions:

Proposed commit message: (wrap lines at 72 characters)
Fix broken link to netlify deployments


Checklist: ☑️

  • Updated the documentation for feature additions and enhancements
  • Added tests for bug fixes or features
  • Linked all related issues
  • No unrelated changes

Reviewer checklist:

Indicate the SEMVER impact of the PR:

  • Major (when you make incompatible API changes)
  • Minor (when you add functionality in a backward compatible manner)
  • Patch (when you make backward compatible bug fixes)

At the end of the review, please label the PR with the appropriate label: r.Major, r.Minor, r.Patch.

Breaking change release note preparation (if applicable):

  • To be included in the release note for any feature that is made obsolete/breaking

Give a brief explanation note about:

  • what was the old feature that was made obsolete
  • any replacement feature (if any), and
  • how the author should modify his website to migrate from the old feature to the replacement feature (if possible).

Copy link
Contributor

@tlylt tlylt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about fixing this link https://github.com/MarkBind/markbind/blob/master/docs/userGuide/gettingStarted.md#L72 as well and make this PR close the targeting issue? Looks like these are the only necessary changes.

@tlylt
Copy link
Contributor

tlylt commented May 1, 2024

As for the init-typical, we should update the repo readme and the repo description

Copy link
Contributor

@EltonGohJH EltonGohJH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@EltonGohJH EltonGohJH merged commit c945986 into MarkBind:master May 1, 2024
@github-actions github-actions bot added the r.Patch Version resolver: increment by 0.0.1 label May 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

r.Patch Version resolver: increment by 0.0.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Netlify .com domain doesn't work for MarkBind Website

3 participants