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.

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

Finalize WIP task for fixing outstanding linter warnings #1497

Merged
merged 1 commit into from
Jun 5, 2024

Conversation

jiridanek
Copy link
Member

  • Fix "F405", # IsNotFlagged may be undefined, or defined from star imports
  • Fix "UP034", # Avoid extraneous parentheses

* Fix "F405", # `IsNotFlagged` may be undefined, or defined from star imports
* Fix "UP034", # Avoid extraneous parentheses
* Fix "FA100", # Missing `from __future__ import annotations`, but uses `typing.Optional`
* "F841", # Local variable `response` is assigned to but never used

Signed-off-by: Jiri Daněk <jdanek@redhat.com>
@jiridanek jiridanek self-assigned this Jun 3, 2024
Copy link

sonarqubecloud bot commented Jun 3, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Contributor

github-actions bot commented Jun 3, 2024

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass %
468 0 0 468 100

@jiridanek jiridanek requested review from apodhrad and kobihk June 4, 2024 18:29
@jiridanek jiridanek enabled auto-merge (squash) June 4, 2024 18:29
@jiridanek jiridanek added misc Miscelaneus (PR will be listed in release-notes) verified This PR has been tested with Jenkins labels Jun 5, 2024
@jiridanek jiridanek merged commit dd15a02 into red-hat-data-services:master Jun 5, 2024
8 checks passed
@jiridanek jiridanek deleted the jd_cleanups branch June 5, 2024 13:18
jgarciao pushed a commit to jgarciao/ods-ci that referenced this pull request Jun 12, 2024
…a-services#1497)

Finalize my WIP task for fixing linter warnings

* Fix "F405", # IsNotFlagged may be undefined, or defined from star imports
* Fix "UP034", # Avoid extraneous parentheses

Signed-off-by: Jiri Daněk <jdanek@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
misc Miscelaneus (PR will be listed in release-notes) verified This PR has been tested with Jenkins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants