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

Separating Pylint and Pytest #27

Merged
merged 3 commits into from
Nov 4, 2024

Conversation

SchrodingersStruggle
Copy link
Collaborator

Separating the Pylint and Pytest file in Workflows.

Linting

  • My code does the linting.

Contents

  • Changed combined workflow file into just a file for linting.
  • Added a separate file for pytesting in workflow.

Labels and Milestones

  • Added organization label.

Reviewer

Any further questions on formatting pull requests can be asked in the issues tab or on discord.
This document can be changed at any time.

Separated pylint and pytest to allow individual testing in order to not produce one error message if either fails.
Isolated linting to have its own individual test to produce a review of a pull request which just summarizes linting result rather than both linting and testing combined.
Copy link
Contributor

@laserlab laserlab left a comment

Choose a reason for hiding this comment

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

Looks good ready to merge

@SchrodingersStruggle SchrodingersStruggle merged commit c25e997 into ubsuny:main Nov 4, 2024
0 of 2 checks passed
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.

2 participants