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 black to fix pipeline #78

Merged
merged 3 commits into from
Apr 3, 2023
Merged

Conversation

hasnain095
Copy link
Contributor

Fixed issue in the pipeline

  • Update GitHub action from v2 to v3
  • Update black from version 20.8b1 to 22.3.0 to fix the error with dependency click

@hasnain095
Copy link
Contributor Author

This hopefully fixes the pipelines issue.

@Kludex
Copy link
Owner

Kludex commented Apr 3, 2023

Can you trigger the pipeline on branches? The "on:" statement needs to be modified 🥺

@hasnain095
Copy link
Contributor Author

You mean on specific branches only, such as 'main' ?

@Kludex
Copy link
Owner

Kludex commented Apr 3, 2023

Yeah, like this: #79

@hasnain095
Copy link
Contributor Author

Its done

@Kludex Kludex changed the title Ci/fix 💚 Upgrade black to fix pipeline Apr 3, 2023
@Kludex Kludex merged commit 841690e into Kludex:main Apr 3, 2023
2 checks passed
@Kludex
Copy link
Owner

Kludex commented Apr 3, 2023

Thanks :)

hasnain095 added a commit to hasnain095/fastapi-microservices that referenced this pull request Apr 4, 2023
💚 Upgrade black to fix pipeline (Kludex#78)
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.

None yet

2 participants