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

[6.13.z] [pre-commit.ci] pre-commit autoupdate #1503

Merged
merged 1 commit into from
Aug 19, 2024
Merged

[6.13.z] [pre-commit.ci] pre-commit autoupdate #1503

merged 1 commit into from
Aug 19, 2024

Conversation

Satellite-QE
Copy link
Contributor

@ogajduse @web-flow
Fix PLW0604
1bd1491 
airgun/entities/host_new.py:25:1: PLW0604 `global` at module level is redundant
   |
23 | from airgun.views.job_invocation import JobInvocationCreateView
24 |
25 | global available_param_types
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ PLW0604
26 | available_param_types = ['string', 'boolean', 'integer', 'real', 'array', 'hash', 'yaml', 'json']
   |

(cherry picked from commit 4921679)
@Satellite-QE Satellite-QE added 6.13.z Auto_Cherry_Picked GHA has automatically cherrypicked this PR No-CherryPick PR doesnt need CherryPick to previous branches AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing labels Aug 19, 2024
@Gauravtalreja1 Gauravtalreja1 merged commit 5ad36bc into 6.13.z Aug 19, 2024
13 of 14 checks passed
@Gauravtalreja1 Gauravtalreja1 deleted the cherry-pick-6.13.z-4921679939c871702b460badf43a86e44c206a73 branch August 19, 2024 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

Assignees
No one assigned
Labels
6.13.z Auto_Cherry_Picked GHA has automatically cherrypicked this PR AutoMerge_Cherry_Picked Automatically merge the PR is PRT and all checks are passing No-CherryPick PR doesnt need CherryPick to previous branches
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Satellite-QE @Gauravtalreja1 @ogajduse