Skip to content

enabled and fixed more pylint warnings#6901

Merged
chrchr-github merged 14 commits intocppcheck-opensource:mainfrom
firewave:pylint-fix
Oct 11, 2024
Merged

enabled and fixed more pylint warnings#6901
chrchr-github merged 14 commits intocppcheck-opensource:mainfrom
firewave:pylint-fix

Conversation

@firewave
Copy link
Copy Markdown
Collaborator

No description provided.


print_ts('packages_nodata: {}'.format(len(packages_nodata)))

print_ts('removing packages with no files to process'.format(len(packages_nodata)))
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Wasn't the intent to print the number of packages (i.e. {} missing)?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

No. It is just c&p error - this is just to indicate the operation is taking place. There are further log message which provide all the remaining details.

@chrchr-github chrchr-github merged commit 4b5935e into cppcheck-opensource:main Oct 11, 2024
@firewave firewave deleted the pylint-fix branch October 11, 2024 09:53
ludviggunne pushed a commit to ludviggunne/cppcheck that referenced this pull request Oct 19, 2024
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.

2 participants