MarkBind / markbind

MarkBind is a tool for generating content-heavy websites from source files in Markdown format
https://markbind.org/
MIT License
134 stars 123 forks source link

Migrate core/index.js to Typescript #2482

Closed yiwen101 closed 3 months ago

yiwen101 commented 3 months ago

What is the purpose of this pull request?

Overview of changes: Working on #1913; quite trivial changes.

Anything you'd like to highlight/discuss:

Testing instructions:

Proposed commit message: (wrap lines at 72 characters) Migrate core/index.js to Typescript


Checklist: :ballot_box_with_check:


Reviewer checklist:

Indicate the SEMVER impact of the PR:

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):

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).
codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 51.11%. Comparing base (c268733) to head (370b3ba).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2482 +/- ## ======================================= Coverage 51.11% 51.11% ======================================= Files 124 124 Lines 5355 5355 Branches 1152 1152 ======================================= Hits 2737 2737 Misses 2328 2328 Partials 290 290 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

kaixin-hc commented 3 months ago

@yiwen101 - please revert the merge commit 😅

yiwen101 commented 3 months ago

@yiwen101 - please revert the merge commit 😅

sorry, I must have accidentally hit "update with merge" someday😅.

itsyme commented 3 months ago

LGTM!

github-actions[bot] commented 3 months ago

@itsyme Each PR must have a SEMVER impact label, please remember to label the PR properly.