Skip to content

fix: Maximum call stack size exceeded#315

Merged
D-Sketon merged 3 commits intohexojs:masterfrom
D-Sketon:fix
Feb 12, 2025
Merged

fix: Maximum call stack size exceeded#315
D-Sketon merged 3 commits intohexojs:masterfrom
D-Sketon:fix

Conversation

@D-Sketon
Copy link
Copy Markdown
Member

@D-Sketon D-Sketon commented Feb 9, 2025

check list

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

Description

Reset the internal configuration used when marked.use to prevent functions from self-referencing

fix #310

Additional information

@coveralls
Copy link
Copy Markdown

coveralls commented Feb 9, 2025

Coverage Status

coverage: 100.0%. remained the same
when pulling 9fc45c6 on D-Sketon:fix
into 9d42f9e on hexojs:master.

@D-Sketon D-Sketon requested a review from a team February 9, 2025 15:34
Comment thread test/index.js Outdated
@uiolee uiolee requested a review from a team February 10, 2025 03:56
@D-Sketon D-Sketon merged commit 4a57485 into hexojs:master Feb 12, 2025
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.

RangeError: Maximum call stack size exceeded

5 participants