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

update globals to v12.1.0 #12296

Merged
merged 1 commit into from Oct 29, 2019
Merged

update globals to v12.1.0 #12296

merged 1 commit into from Oct 29, 2019

Conversation

UziTech
Copy link
Contributor

@UziTech UziTech commented Sep 21, 2019

What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[x] Other, please explain:

Update dependency

What changes did you make? (Give an overview)

Update globals dependency to v12.1.0 Here are the changes

Most notably for me is the addition of globals to the jasmine environment:
expectAsync and spyOnAllFunctions

Is there anything you'd like reviewers to focus on?

No

@jsf-clabot
Copy link

jsf-clabot commented Sep 21, 2019

CLA assistant check
All committers have signed the CLA.

@eslint-deprecated eslint-deprecated bot added the triage An ESLint team member will look at this issue soon label Sep 21, 2019
@kaicataldo kaicataldo added evaluating The team will evaluate this issue to decide whether it meets the criteria for inclusion upgrade This change is related to a dependency upgrade and removed triage An ESLint team member will look at this issue soon labels Sep 21, 2019
@kaicataldo
Copy link
Member

kaicataldo commented Sep 21, 2019

sindresorhus/globals#143 looks like the only change that would affect existing globals. It looks like it'll lead to fewer errors, though, so I think it's fine to release this as a semver-minor bug fix.

@kaicataldo kaicataldo added the enhancement This change enhances an existing feature of ESLint label Sep 21, 2019
@UziTech
Copy link
Contributor Author

UziTech commented Oct 27, 2019

Is there anything blocking this?

@ilyavolodin
Copy link
Member

ilyavolodin commented Oct 27, 2019

No, sorry, we lost track of this PR. Thanks for the reminder. We are currently in merge freeze due to possibility of patch release, but once we are out of it, we'll merge this in.

@ilyavolodin ilyavolodin self-assigned this Oct 27, 2019
@ilyavolodin ilyavolodin added accepted There is consensus among the team that this change meets the criteria for inclusion and removed evaluating The team will evaluate this issue to decide whether it meets the criteria for inclusion labels Oct 29, 2019
@ilyavolodin ilyavolodin merged commit 98e1d50 into eslint:master Oct 29, 2019
@eslint-deprecated eslint-deprecated bot locked and limited conversation to collaborators Apr 28, 2020
@eslint-deprecated eslint-deprecated bot added the archived due to age This issue has been archived; please open a new issue for any further discussion label Apr 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accepted There is consensus among the team that this change meets the criteria for inclusion archived due to age This issue has been archived; please open a new issue for any further discussion enhancement This change enhances an existing feature of ESLint upgrade This change is related to a dependency upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants