-
-
Notifications
You must be signed in to change notification settings - Fork 48
chore: use context.cwd
#1176
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
chore: use context.cwd
#1176
Conversation
🦋 Changeset detectedLatest commit: 80dd3f1 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
fb385e2
to
a7267b7
Compare
Try the Instant Preview in Online PlaygroundInstall the Instant Preview to Your Local
Published Instant Preview Packages:
|
weirdly this CI failure doesn't seem related but im not sure why else it'd be happening 🤔 |
Switches to using `context.cwd` since all ESLint versions we support now have it. The previously used `getCwd` helper falls back to `process.cwd()` (which ESLint itself doesn't do). So in the few places we relied on that, a fall back to `process.cwd()` has been added.
it was me being blind. we still depend on the compat package for the rule tester |
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.
Thank you!
@43081j Will you make a 1 more PR for |
i was looking at that, i feel like we should be able to drop most of it (or all of it?) i'll see where i get with it 👍 |
Switches to using
context.cwd
since all ESLint versions we support now have it.The previously used
getCwd
helper falls back toprocess.cwd()
(which ESLint itself doesn't do). So in the few places we relied on that, a fall back toprocess.cwd()
has been added.This is the last remaining compat function, so we have also dropped theeslint-compat-utils
dependency.