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 black et al #4155

Merged
merged 4 commits into from
May 26, 2021
Merged

Update black et al #4155

merged 4 commits into from
May 26, 2021

Conversation

bjlittle
Copy link
Member

🚀 Pull Request

Description

This PR updates black and flake8 pre-commit hooks scripts, and pre-commit-hooks itself.

Also plays forward the bump in black version to the rest of iris, with minor knock-on reformatting to conform to the changed standard.


Consult Iris pull request check list

@lbdreyer
Copy link
Member

Found another mention of the black version that you could consider updating: https://github.com/SciTools/iris/blob/master/noxfile.py#L193

@bjlittle
Copy link
Member Author

@lbdreyer Jeez... it's flippin everywhere 😱

I'll fix that now...

@lbdreyer lbdreyer merged commit 956df99 into SciTools:master May 26, 2021
@bjlittle bjlittle deleted the update-black-et-al branch May 27, 2021 20:20
tkknight added a commit to tkknight/iris that referenced this pull request Jun 4, 2021
* master:
  refactor setup.py to setup.cfg (SciTools#4168)
  update docs pypi release (SciTools#4173)
  Update CI environment lockfiles (SciTools#4137)
  update CONTRIBUTING.md (SciTools#4165)
  RTD support link update (SciTools#4166)
  drop py36 support (SciTools#4163)
  github issues contact link for discussions (SciTools#4164)
  Bump black version (SciTools#4162)
  Stop CI from clobbering commits on lockfile updates (SciTools#4157)
  [pre-commit.ci] pre-commit autoupdate (SciTools#4161)
  Add a method to return a CubeList from CubeList.copy() (SciTools#4094)
  Update black et al (SciTools#4155)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants