Skip to content

Conversation

LincZero
Copy link
Contributor

Before submitting the PR, please make sure you do the following

  • Read the Contributing Guidelines.
  • Provide a description in this PR that addresses what the PR is solving. If this PR is going to solve an existing issue, please reference the issue (e.g. close #123).

What is the purpose of this pull request?

  • Bug fix
  • New feature
  • Other

Description

close #445

Screenshots

Before

After

@LincZero LincZero requested a review from Mister-Hope April 30, 2025 14:56
@Mister-Hope
Copy link
Member

CI is broken as well, I think you might forget a successful pnpm i

@LincZero
Copy link
Contributor Author

Both my local pnpm i and pnpm build were successful before committing. I think the file was missed just before committing

@LincZero LincZero requested a review from Mister-Hope April 30, 2025 15:07
@Mister-Hope
Copy link
Member

Ensure corepack is enabled in your device, the override field is not being handled correctly.

@LincZero
Copy link
Contributor Author

LincZero commented Apr 30, 2025

I don't understand.

> corepack enable
Internal Error: EPERM: operation not permitted, open 'G:\Soft\Dev\Web\Js_Node\pnpm'
Error: EPERM: operation not permitted, open 'G:\Soft\Dev\Web\Js_Node\pnpm'

I used another method. Directly trace back the lock file, and then re pnpm i on github codespace.

Copy link
Member

@Mister-Hope Mister-Hope left a comment

Choose a reason for hiding this comment

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

linter test failed

@LincZero LincZero requested a review from Mister-Hope April 30, 2025 16:57
@coveralls
Copy link

coveralls commented Apr 30, 2025

Pull Request Test Coverage Report for Build 14759999858

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 59.561%

Totals Coverage Status
Change from base Build 14751218979: 0.0%
Covered Lines: 1362
Relevant Lines: 2063

💛 - Coveralls

@Mister-Hope Mister-Hope enabled auto-merge (squash) May 1, 2025 04:47
@Mister-Hope Mister-Hope merged commit 8489262 into vuepress:main May 1, 2025
15 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.

[Feature request] markdown-image use better syntax
3 participants