Skip to content

Add checksum for py-virtualenv 20.16.4#33154

Merged
adamjstewart merged 4 commits intospack:developfrom
iarspider:update-py-virtualenv
Oct 11, 2022
Merged

Add checksum for py-virtualenv 20.16.4#33154
adamjstewart merged 4 commits intospack:developfrom
iarspider:update-py-virtualenv

Conversation

@iarspider
Copy link
Copy Markdown
Contributor

No description provided.

@iarspider
Copy link
Copy Markdown
Contributor Author

@spackbot fix style

@spackbot-app
Copy link
Copy Markdown

spackbot-app bot commented Oct 10, 2022

Let me see if I can fix that for you!

@spackbot-app
Copy link
Copy Markdown

spackbot-app bot commented Oct 10, 2022

I was able to run spack style --fix for you!

spack style --fix
==> Running style checks on spack
  selected: isort, mypy, black, flake8
==> Modified files
  var/spack/repos/builtin/packages/py-virtualenv/package.py
==> Running isort checks
  isort checks were clean
==> Running mypy checks
Success: no issues found in 558 source files
  mypy checks were clean
==> Running black checks
reformatted var/spack/repos/builtin/packages/py-virtualenv/package.py
All done! ✨ 🍰 ✨
1 file reformatted.
  black checks were clean
==> Running flake8 checks
  flake8 checks were clean
==> spack style checks were clean
Keep in mind that I cannot fix your flake8 or mypy errors, so if you have any you'll need to fix them and update the pull request. If I was able to push to your branch, if you make further changes you will need to pull from your updated branch before pushing again.

I've updated the branch with isort fixes.

Copy link
Copy Markdown
Contributor

@tldahlgren tldahlgren left a comment

Choose a reason for hiding this comment

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

Confirmed sha256.

@tldahlgren tldahlgren self-assigned this Oct 10, 2022
@adamjstewart adamjstewart self-assigned this Oct 10, 2022
@iarspider
Copy link
Copy Markdown
Contributor Author

Can I ask for #33047 to be merged now, since this PR kind of depends on it?

@adamjstewart adamjstewart enabled auto-merge (squash) October 10, 2022 17:09
@adamjstewart adamjstewart merged commit be293ce into spack:develop Oct 11, 2022
@adamjstewart
Copy link
Copy Markdown
Member

@alalazo this PR was failing audit tests (missing dep version) but still auto-merged. Any idea what happened here?

@adamjstewart
Copy link
Copy Markdown
Member

Btw, if #33047 ever passes CI, this issue will be solved in develop.

@adamjstewart
Copy link
Copy Markdown
Member

Are our audit tests not required for CI to pass?

@alalazo
Copy link
Copy Markdown
Member

alalazo commented Oct 11, 2022

They should be, let me check

luke-dt pushed a commit to dantaslab/spack that referenced this pull request Oct 12, 2022
* Add checksum for py-virtualenv 20.16.4

* [@spackbot] updating style on behalf of iarspider

* Update package.py

* Update package.py

Co-authored-by: iarspider <[email protected]>
@iarspider iarspider deleted the update-py-virtualenv branch October 12, 2022 11:29
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.

4 participants