Skip to content

Conversation

DavidFair
Copy link
Collaborator

Adds CI and CD for rally in-line with our other source directories

This makes it clear which file is the Python and shell script to future
developers and users
Runs the Black formatter against the rally-tester script to make it
uniform with other scripts
@DavidFair DavidFair force-pushed the Ci_cd_rally branch 3 times, most recently from ded07fa to 3451e95 Compare July 13, 2023 10:00
Adds Pylint and shell check for common errors against out rally script.
This commit does not resolve any warnings at this stage
@DavidFair DavidFair force-pushed the Ci_cd_rally branch 7 times, most recently from ec9cce2 to a6fc8fa Compare July 13, 2023 10:15
@DavidFair DavidFair marked this pull request as ready for review July 13, 2023 10:15
@DavidFair DavidFair force-pushed the Ci_cd_rally branch 5 times, most recently from 80bfc42 to dab23df Compare July 13, 2023 10:32
Resolves the warnings from Pylint and shellcheck
Copy link
Collaborator

@apdibbo apdibbo left a comment

Choose a reason for hiding this comment

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

Seems to still work so approving

@apdibbo apdibbo merged commit a54746d into master Jul 13, 2023
@khalford khalford deleted the Ci_cd_rally branch September 19, 2024 13:07
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