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

Add a minimal CI workflow #114

Merged
merged 3 commits into from Mar 9, 2024

Conversation

pnahratow
Copy link
Contributor

@pnahratow pnahratow commented Mar 7, 2024

This change adds a tiny CI workflow to keep the python files clean.
I made the initial changes to the py files which mostly was inconsistent whitespaces and some minor issues.

The pipelines runs only on pull requests at moment and in the future could be expanded to check that your usual release build workflows runs without issues.

PS: I have excluded docs/.etherpad-backups from the checks since I assume this is an unmaintained file.
Also since github actions haven't been configured for this repository before the CI run won't be triggered due to security but should run on the next PR.

Copy link
Collaborator

@MotoLegacy MotoLegacy left a comment

Choose a reason for hiding this comment

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

Looks great! It's worth noting we plan to move away from Python eventually(tm), but in the event of further script modification, a standardized linter would do great :)

@MissLavender-LQ MissLavender-LQ merged commit bbe8e5a into MissLavender-LQ:main Mar 9, 2024
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

3 participants