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

Upgrade GitHub Actions #83

Merged
merged 5 commits into from
Aug 25, 2022
Merged

Upgrade GitHub Actions #83

merged 5 commits into from
Aug 25, 2022

Conversation

cclauss
Copy link
Contributor

@cclauss cclauss commented Aug 22, 2022

No description provided.

.github/workflows/deploy.yaml Outdated Show resolved Hide resolved
.github/workflows/deploy.yaml Outdated Show resolved Hide resolved
Co-authored-by: Nicola Soranzo <[email protected]>
Copy link
Collaborator

@nsoranzo nsoranzo left a comment

Choose a reason for hiding this comment

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

Thanks!

@cclauss
Copy link
Contributor Author

cclauss commented Aug 24, 2022

Flake8 does not like #53

@nsoranzo
Copy link
Collaborator

Flake8 does not like #53

Not exactly, E275 has recently been added in pycodestyle 2.9.0 https://pycodestyle.pycqa.org/en/latest/developer.html#id3

If you drop the last commit disabling E275, we can merge this and address the flake8 errors separately.

@nsoranzo
Copy link
Collaborator

If you drop the last commit disabling E275, we can merge this and address the flake8 errors separately.

Done in #84 .

@nsoranzo nsoranzo merged commit 981890f into bxlab:main Aug 25, 2022
@nsoranzo
Copy link
Collaborator

Thanks @cclauss !

@cclauss cclauss deleted the patch-1 branch August 25, 2022 03:05
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