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

Fix: Use caret range for requireindex in template #82

Merged
merged 1 commit into from Jul 6, 2021

Conversation

bmish
Copy link
Sponsor Member

@bmish bmish commented Jun 29, 2021

Avoid unnecessarily constraining the dependency versions which prevents users from receiving patch/minor updates.

@nzakas
Copy link
Member

nzakas commented Jul 2, 2021

Looks like there are conflicts with the master branch.

@bmish
Copy link
Sponsor Member Author

bmish commented Jul 2, 2021

Fixed conflicts. I'll open up several more PRs after this.

Copy link
Contributor

@snitin315 snitin315 left a comment

Choose a reason for hiding this comment

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

LGTM, Thanks!

@snitin315 snitin315 added accepted and removed triage labels Jul 2, 2021
@nzakas
Copy link
Member

nzakas commented Jul 2, 2021

Heh. Looks like package-lock.json keeps getting conflicts due to other PRs being merged.

@bmish
Copy link
Sponsor Member Author

bmish commented Jul 3, 2021

Fixed.

@bmish bmish changed the title Fix: Use caret ranges for a few dependencies Fix: Use caret range for requireindex in template Jul 3, 2021
Avoid unnecessarily constraining the dependency versions which prevents users from receiving patch/minor updates.
@bmish
Copy link
Sponsor Member Author

bmish commented Jul 6, 2021

@nzakas this one is ready.

@snitin315 snitin315 merged commit 6d3e1e2 into eslint:main Jul 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants