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.

Sign up for GitHub

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

Jump to bottom

use the runner.yaml constant #5670

Merged
merged 3 commits into from
Jul 19, 2022
Merged

use the runner.yaml constant #5670

merged 3 commits into from
Jul 19, 2022

Conversation

cognifloyd
Copy link
Member

A very minor refactor.

I noticed that we had a constant and weren't using it. So, let's use it.

st2/st2common/st2common/constants/runners.py

Lines 40 to 41 in 8496bb2

# Manifest file name for runners
MANIFEST_FILE_NAME = "runner.yaml"

@cognifloyd cognifloyd added this to the 3.8.0 milestone Jul 16, 2022
@cognifloyd cognifloyd added no changelog No Changelog.rst needed for this PR refactor labels Jul 16, 2022
@cognifloyd cognifloyd requested a review from a team July 16, 2022 22:32
@pull-request-size pull-request-size bot added the size/XS PR that changes 0-9 lines. Quick fix/merge. label Jul 16, 2022
@cognifloyd cognifloyd self-assigned this Jul 16, 2022
rush-skills
rush-skills approved these changes Jul 17, 2022
View reviewed changes
Copy link
Member

@rush-skills rush-skills left a comment

Choose a reason for hiding this comment

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

Good catch 👍🏼

@cognifloyd cognifloyd added no changelog No Changelog.rst needed for this PR and removed no changelog No Changelog.rst needed for this PR labels Jul 17, 2022
amanda11
amanda11 approved these changes Jul 19, 2022
View reviewed changes
Copy link
Contributor

@amanda11 amanda11 left a comment

Choose a reason for hiding this comment

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

LGTM

@cognifloyd cognifloyd merged commit 942ea87 into master Jul 19, 2022
@cognifloyd cognifloyd deleted the use-runner-constant branch July 19, 2022 15:20
@cognifloyd cognifloyd mentioned this pull request Aug 2, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amanda11 amanda11 amanda11 approved these changes

@rush-skills rush-skills rush-skills approved these changes

Assignees

@cognifloyd cognifloyd

Labels
no changelog No Changelog.rst needed for this PR pantsbuild refactor size/XS PR that changes 0-9 lines. Quick fix/merge.
Projects
None yet
Milestone
3.8.0
Development

Successfully merging this pull request may close these issues.
3 participants
@cognifloyd @rush-skills @amanda11