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

global variable string change #2

Merged
merged 1 commit into from
Apr 5, 2022
Merged

global variable string change #2

merged 1 commit into from
Apr 5, 2022

Conversation

jkanem
Copy link
Contributor

@jkanem jkanem commented Mar 31, 2022

Changed global variable executor_plugin_name -> EXECUTOR_PLUGIN_NAME in executors to conform with PEP8

@jkanem jkanem changed the title global varialbe string change global variable string change Mar 31, 2022
@jkanem jkanem added the improvements / style Improvement in style or design label Apr 4, 2022
@jkanem jkanem requested review from FyzHsn and kessler-frost April 4, 2022 19:47
@jkanem jkanem merged commit 1751ac1 into main Apr 5, 2022
@jkanem jkanem deleted the cova_updates branch April 5, 2022 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FyzHsn FyzHsn FyzHsn approved these changes

@kessler-frost kessler-frost Awaiting requested review from kessler-frost

Assignees
No one assigned
Labels
improvements / style Improvement in style or design
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jkanem @FyzHsn