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

Create black pipeline to check PRs #54

Merged
merged 4 commits into from
Nov 9, 2023

Conversation

parisosuch-dev
Copy link
Member

Pipelines baby -- pipelines.

@parisosuch-dev
Copy link
Member Author

love how black checks the PR that creates the black pipeline

@parisosuch-dev
Copy link
Member Author

gimme a sec to fix this @brennanfreeze

@parisosuch-dev
Copy link
Member Author

ill fix this later -- just leave PR open for now 🤷🏼

@parisosuch-dev parisosuch-dev self-assigned this Nov 8, 2023
@parisosuch-dev
Copy link
Member Author

went ahead and just added an auto formatter for PRs instead of checking if the code is formatted to PEP8 styling. This works out better imo, making it to where people who contribute do not need to know about installing black formatter.

@brennanfreeze
Copy link
Member

Looks good, definitely needed this

@parisosuch-dev parisosuch-dev merged commit 96ef041 into main Nov 9, 2023
1 check passed
@parisosuch-dev parisosuch-dev deleted the feature/add-black-formatter-pipeline branch November 9, 2023 04:20
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