Skip to content
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(lint): Humanise eslint error for accessing process.env #8638

Merged
merged 2 commits into from
Jun 16, 2023

Conversation

dac09
Copy link
Collaborator

@dac09 dac09 commented Jun 16, 2023

As title, just changes the error message.

@dac09 dac09 requested a review from Tobbe June 16, 2023 08:44
@dac09
Copy link
Collaborator Author

dac09 commented Jun 16, 2023

@Tobbe just changed the message on the eslint plugin. Let me know what you think.

I think the old message would only make sense to AST people, even if this one is less correct.

@Tobbe Tobbe added the release:chore This PR is a chore (means nothing for users) label Jun 16, 2023
@Tobbe Tobbe enabled auto-merge (squash) June 16, 2023 08:48
@Tobbe Tobbe merged commit d987b68 into redwoodjs:main Jun 16, 2023
28 checks passed
@redwoodjs-bot redwoodjs-bot bot added this to the next-release milestone Jun 16, 2023
@jtoar jtoar modified the milestones: next-release, v6.0.0 Jun 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:chore This PR is a chore (means nothing for users)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants