Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: refactor types #5344

Merged
merged 7 commits into from
Jan 18, 2024
Merged

refactor: refactor types #5344

merged 7 commits into from
Jan 18, 2024

Conversation

D-Sketon
Copy link
Member

@D-Sketon D-Sketon commented Nov 5, 2023

What does it do?

Screenshots

Pull request tasks

  • Add test cases for the changes.
  • Passed the CI test.

This comment was marked as outdated.

@D-Sketon D-Sketon marked this pull request as draft November 5, 2023 15:04
@D-Sketon D-Sketon marked this pull request as ready for review November 9, 2023 08:14
@D-Sketon
Copy link
Member Author

D-Sketon commented Nov 9, 2023

should we change test files to ts ?

@D-Sketon D-Sketon marked this pull request as draft November 9, 2023 12:01
@D-Sketon
Copy link
Member Author

D-Sketon commented Nov 9, 2023

found some type errors when changing test files to ts

@uiolee
Copy link
Member

uiolee commented Dec 21, 2023

should we change test files to ts ?

I guess that's acceptable. That way 100% ts can be achieved. But I believe this should be done in another PR

pic
pic

Copy link

github-actions bot commented Jan 9, 2024

How to test

git clone -b v7.0.0_9 https://github.com/D-Sketon/hexo.git
cd hexo
npm install
npm test

@D-Sketon D-Sketon marked this pull request as ready for review January 9, 2024 04:11
@coveralls
Copy link

coveralls commented Jan 11, 2024

Pull Request Test Coverage Report for Build 7502963918

Warning: This coverage report may be inaccurate.

We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.006%) to 99.546%

Totals Coverage Status
Change from base Build 7461275554: 0.006%
Covered Lines: 9216
Relevant Lines: 9258

💛 - Coveralls

uiolee
uiolee previously approved these changes Jan 11, 2024
lib/extend/renderer.ts Outdated Show resolved Hide resolved
SukkaW
SukkaW previously approved these changes Jan 12, 2024
@D-Sketon D-Sketon dismissed stale reviews from SukkaW and uiolee via 1916cab January 12, 2024 13:42
Co-authored-by: Sukka <isukkaw@gmail.com>
@uiolee uiolee merged commit 86350d9 into hexojs:master Jan 18, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants