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

Ignore: make linters angry #35

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

praiskup
Copy link
Member

@praiskup praiskup commented Oct 1, 2024

@praiskup praiskup changed the title Make linters angry Make linters angry - demonstrate https://github.com/fedora-copr/vcs-diff-lint-action/issues/19 Oct 1, 2024
@@ -292,7 +292,7 @@

path = os.getcwd()
while True:
log.info("checking for projectdir: %s", path)
log.info("checking for projectdir: {}".format(path))

Check warning

Code scanning / vcs-diff-lint

_Worker._analyze_projectdir: Use lazy % formatting in logging functions Warning

_Worker._analyze_projectdir: Use lazy % formatting in logging functions
@@ -292,7 +292,7 @@

path = os.getcwd()
while True:
log.info("checking for projectdir: %s", path)
log.info("checking for projectdir: {}".format(path))

Check warning

Code scanning / vcs-diff-lint

_Worker._analyze_projectdir: Formatting a regular string which could be an f-string Warning

_Worker._analyze_projectdir: Formatting a regular string which could be an f-string
@praiskup praiskup changed the title Make linters angry - demonstrate https://github.com/fedora-copr/vcs-diff-lint-action/issues/19 Ignore: make linters angry Oct 1, 2024
@praiskup
Copy link
Member Author

praiskup commented Oct 1, 2024

This one new issue not reported to UI:

Error: RUFF_WARNING:
vcs-diff-lint:11:8: F401[unused-import]: `glob` imported but unused

@praiskup
Copy link
Member Author

praiskup commented Oct 1, 2024

Available here, but I had to guess the link:
https://github.com/fedora-copr/vcs-diff-lint/security/code-scanning/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant