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

testing and linting updates #305

Merged
merged 9 commits into from
Aug 26, 2024

Conversation

zackbradys
Copy link
Member

Please check below, if the PR fulfills these requirements:

  • Commit(s) and code follow the repositories guidelines.
  • Test(s) have been added or updated to support these change(s).
  • Doc(s) have been added or updated to support these change(s).

Associated Links:

  • N/A

Types of Changes:

  • Feature

Proposed Changes:

  • Updated unittest.yaml to tests.yaml to improve the overall testing and linting of hauler
  • Added comprehensive unit and integration tests that build the binaries and test the newly built binaries

Verification/Testing of Changes:

  • Unfortunately, it is not very easy to simulate GitHub Actions. You are able to test by manually running each of the workflows, jobs, steps, and commands locally to verify they work
  • I completed this on multiple ubuntu machines

Additional Context:

  • N/A

@zackbradys zackbradys self-assigned this Aug 23, 2024
@zackbradys zackbradys requested a review from a team August 23, 2024 05:36
@zackbradys zackbradys added enhancement New feature or request size/L Denotes an issue/PR requiring a relatively large amount of work labels Aug 23, 2024
@zackbradys zackbradys added this to the Hauler v1.1.0 milestone Aug 23, 2024
@zackbradys zackbradys removed the request for review from a team August 23, 2024 13:50
@zackbradys zackbradys force-pushed the testing-linting-updates branch 6 times, most recently from c51327f to f974802 Compare August 26, 2024 04:41
@zackbradys zackbradys requested a review from a team August 26, 2024 13:46
@zackbradys zackbradys merged commit cb81823 into hauler-dev:main Aug 26, 2024
2 checks passed
@zackbradys zackbradys deleted the testing-linting-updates branch August 26, 2024 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request size/L Denotes an issue/PR requiring a relatively large amount of work
Projects
Status: Resolved
Development

Successfully merging this pull request may close these issues.

2 participants