Skip to content

feat(rspeedy/core): apply Rsdoctor linter.rules['ecma-version-check']#282

Merged
colinaaa merged 5 commits intolynx-family:mainfrom
colinaaa:colin/0321/rsdoctor
Mar 21, 2025
Merged

feat(rspeedy/core): apply Rsdoctor linter.rules['ecma-version-check']#282
colinaaa merged 5 commits intolynx-family:mainfrom
colinaaa:colin/0321/rsdoctor

Conversation

@colinaaa
Copy link
Collaborator

Summary

Apply a default rule option which will only warn syntax beyond ES2019.

This is a following up to #186.

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@changeset-bot
Copy link

changeset-bot bot commented Mar 21, 2025

🦋 Changeset detected

Latest commit: d6b14c6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

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

@codecov
Copy link

codecov bot commented Mar 21, 2025

Codecov Report

Attention: Patch coverage is 77.77778% with 4 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ckages/rspeedy/core/src/plugins/rsdoctor.plugin.ts 77.77% 4 Missing ⚠️

📢 Thoughts on this report? Let us know!

@colinaaa colinaaa marked this pull request as ready for review March 21, 2025 09:30
Copy link
Collaborator

@gaoachao gaoachao left a comment

Choose a reason for hiding this comment

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

LGTM!

@colinaaa colinaaa added this pull request to the merge queue Mar 21, 2025
Merged via the queue into lynx-family:main with commit 0e75123 Mar 21, 2025
15 of 16 checks passed
@colinaaa colinaaa deleted the colin/0321/rsdoctor branch March 21, 2025 10:15
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.

2 participants