Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Remove unused constant#6326

Merged
zrhoffman merged 1 commit intoapache:masterfrom
jrushford:remove-unused-constant
Nov 4, 2021
Merged

Remove unused constant#6326
zrhoffman merged 1 commit intoapache:masterfrom
jrushford:remove-unused-constant

Conversation

@jrushford
Copy link
Copy Markdown
Contributor

Removes an unused string constant.


Which Traffic Control components are affected by this PR?

  • Traffic Control Health Client (tc-health-client)

What is the best way to verify this PR?

@jrushford jrushford closed this Nov 3, 2021
@jrushford jrushford force-pushed the remove-unused-constant branch from cd7e9cd to 5cc9fd9 Compare November 3, 2021 15:56
@jrushford jrushford deleted the remove-unused-constant branch November 3, 2021 15:58
@jrushford jrushford reopened this Nov 3, 2021
@zrhoffman
Copy link
Copy Markdown
Member

In order for CI to pass, #6326 needs to be rebased onto the apache:master branch now that the CI fix in #6329 is merged:

git remote add apache git@github.com:apache/trafficcontrol.git
git fetch apache
git rebase apache
git push your-fork your-branch --force

@jrushford jrushford force-pushed the remove-unused-constant branch from a176f01 to 585a05f Compare November 4, 2021 17:21
Copy link
Copy Markdown
Member

@zrhoffman zrhoffman left a comment

Choose a reason for hiding this comment

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

Yep that constant was unused

@zrhoffman zrhoffman merged commit b9a4e25 into apache:master Nov 4, 2021
@zrhoffman zrhoffman added this to the 6.0.1 milestone Nov 5, 2021
zrhoffman pushed a commit that referenced this pull request Nov 5, 2021
(cherry picked from commit b9a4e25)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants