Skip to content

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.

# 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
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
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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

Development

Successfully merging this pull request may close these issues.

4 participants