-
-
Notifications
You must be signed in to change notification settings - Fork 35
fix: take cwd
into cache key account
#577
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
Conversation
🦋 Changeset detectedLatest commit: 698dfee The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR fixes caching issues by incorporating the current working directory (cwd) into cache key generation and standardizes the use of the import.meta.url value.
- Introduces a constant for import.meta.url to improve consistency in module requiring across files.
- Updates the cache key in the remark processor to include cwd and a delimiter to differentiate paths.
- Removes the redundant "module-sync" export in package.json.
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
packages/eslint-plugin-mdx/src/helpers.ts | Uses a constant (importMetaUrl) for cjsRequire for improved clarity. |
packages/eslint-plugin-mdx/package.json | Removes the deprecated "module-sync" field. |
packages/eslint-mdx/src/worker.ts | Updates getRemarkConfig and getRemarkProcessor to consider cwd in cache. |
packages/eslint-mdx/src/helpers.ts | Applies the same importMetaUrl change for cjsRequire usage. |
packages/eslint-mdx/package.json | Removes the deprecated "module-sync" field. |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #577 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 20 20
Lines 270 272 +2
Branches 46 46
=========================================
+ Hits 270 272 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Initial checklist
Description of changes
follow #571