Skip to content

rename some additional incorrectly prefixed environment variables#434

Merged
MoralCode merged 2 commits into
mainfrom
rename/env_2
Jul 2, 2026
Merged

rename some additional incorrectly prefixed environment variables#434
MoralCode merged 2 commits into
mainfrom
rename/env_2

Conversation

@MoralCode

Copy link
Copy Markdown
Contributor

Description
This is a small PR to address a couple more env vars in the code that still retained the old prefix.

It follows up/should have been part of #311

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Generative AI disclosure

Please select one option:

  • This contribution was NOT assisted or created by Generative AI tools.
  • This contribution was assisted or created by Generative AI tools.

If AI tools were used, please provide details below:
- What tools were used?
- How were these tools used?
- Did you review these outputs before submitting this PR?

Signed-off-by: Adrian Edwards <adredwar@redhat.com>
@@ -32,7 +32,7 @@

from keyman.KeyClient import KeyClient, KeyPublisher

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

[pylint] reported by reviewdog 🐶
W0611: Unused KeyClient imported from keyman.KeyClient (unused-import)

@MoralCode MoralCode changed the title rename some additional incorrectly prefixed variables rename some additional incorrectly prefixed environment variables Jul 2, 2026
@MoralCode MoralCode added this to the v1.1 Migration Release milestone Jul 2, 2026
Signed-off-by: Adrian Edwards <adredwar@redhat.com>
@MoralCode MoralCode added the ready ready to merge, likely waiting for unit tests or something label Jul 2, 2026
@MoralCode

Copy link
Copy Markdown
Contributor Author

This doesn't seem to cause issues

@MoralCode MoralCode merged commit 8bd2f1f into main Jul 2, 2026
16 checks passed
@MoralCode MoralCode deleted the rename/env_2 branch July 2, 2026 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready ready to merge, likely waiting for unit tests or something

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant