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

refactor(parsers): use const for vars data and sort #4202

Merged
merged 3 commits into from
Mar 22, 2022

Conversation

trivikr
Copy link
Contributor

@trivikr trivikr commented Mar 9, 2022

What's the problem this PR addresses?

The variables data and sort in parsers can be declared as const, and assigned values in a conditional operator using TypeScript tuples.

How did you fix it?

Use const for variables data and sort

Checklist

  • I have set the packages that need to be released for my changes to be effective.
  • I will check that all automated PR checks pass before the PR gets reviewed.

@merceyz merceyz changed the title chore(parsers): use const for vars data and sort refactor(parsers): use const for vars data and sort Mar 9, 2022
@trivikr trivikr changed the title refactor(parsers): use const for vars data and sort chore(parsers): use const for vars data and sort Mar 13, 2022
@trivikr trivikr force-pushed the trivikr/parsers/const-data-sort branch from bcd3a57 to ea96f07 Compare March 20, 2022 17:25
@merceyz merceyz changed the title chore(parsers): use const for vars data and sort refactor(parsers): use const for vars data and sort Mar 22, 2022
@merceyz merceyz changed the title refactor(parsers): use const for vars data and sort refactor(parsers): use const for vars data and sort Mar 22, 2022
@merceyz merceyz enabled auto-merge (squash) March 22, 2022 23:45
@merceyz merceyz merged commit 23c8286 into yarnpkg:master Mar 22, 2022
@trivikr trivikr deleted the trivikr/parsers/const-data-sort branch March 22, 2022 23:47
merceyz pushed a commit that referenced this pull request Oct 21, 2023
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