Skip to content

Conversation

@dimaslanjaka
Copy link
Contributor

@dimaslanjaka dimaslanjaka commented Oct 5, 2024

What does it do?

  • Apply hexo context for this callback, useful for plugin written with typescript. Sample
  • Fix some missmatch type (fail compile fixed)

Screenshots

image
image

based on https://hexo.io/api/helper#How-do-I-use-another-registered-helper-in-my-custom-helper
image

Pull request tasks

  • Add test cases for the changes.
  • Passed the CI test.
@github-actions
Copy link

github-actions bot commented Oct 5, 2024

How to test

git clone -b fix-helper-context https://github.com/dimaslanjaka/hexo.git
cd hexo
npm install
npm test
@dimaslanjaka
Copy link
Contributor Author

dimaslanjaka commented Oct 5, 2024

@uiolee @SukkaW review pls, any miss/wrong ?

@SukkaW SukkaW requested a review from a team December 10, 2024 15:29
Copy link
Member

@SukkaW SukkaW left a comment

Choose a reason for hiding this comment

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

LGTM.

@hexojs/core What do you think?

Copy link
Member

@renbaoshuo renbaoshuo left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@SukkaW SukkaW merged commit ebe92ba into hexojs:master Dec 11, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants