Skip to content

build(deps-dev): bump ruff from 0.15.20 to 0.16.4. Closes #60 - #61

Merged
opbot-xd merged 1 commit into
developfrom
chore/bump-ruff-0.16.4
Aug 28, 2026
Merged

build(deps-dev): bump ruff from 0.15.20 to 0.16.4. Closes #60#61
opbot-xd merged 1 commit into
developfrom
chore/bump-ruff-0.16.4

Conversation

@opbot-xd

Copy link
Copy Markdown
Collaborator

Description

This PR bumps the development dependency ruff from version 0.15.20 to 0.16.4, updates .github/.pre-commit-config.yaml to v0.16.4, and adds CPY001 (flake8-copyright) to the ignored lint rules in pyproject.toml.

Because select = ["ALL"] is enabled for Ruff, upgrading to 0.16.4 triggered CPY001 (missing copyright header) on all 13 Python source and test files. Since this project uses a repository-level MIT license without per-file copyright headers, CPY001 is ignored.

Related issues

Type of change

  • Bug fix (non-breaking change which fixes an issue).
  • New feature (non-breaking change which adds functionality).
  • Breaking change (fix or feature that would cause existing functionality to not work as expected).
  • Chore (refactoring, dependency updates, CI/CD changes, code cleanup, docs-only changes).

Checklist

Formalities

  • I chose an appropriate title for the pull request in the form: <feature name>. Closes #999
    - Release PRs (develop → main): Title must be the version number (e.g. 0.2.0)
  • My branch is based on develop.
  • The pull request is for the branch develop.
    - Exception: Release PRs target main.
  • I have reviewed and verified any LLM-generated code included in this PR.

Docs and tests

  • I documented my code changes with docstrings and/or comments.
  • Linter (Ruff) gave 0 errors.
  • I have added tests for the feature/bug I solved.
  • All the tests gave 0 errors.

@opbot-xd
opbot-xd force-pushed the chore/bump-ruff-0.16.4 branch from 453a414 to 4d549b1 Compare August 28, 2026 05:28
Comment thread pyproject.toml
"ISC001", # Conflicts with formatter
"D100", # Missing docstring in public module
"D104", # Missing docstring in public package
"CPY001", # Missing copyright notice at top of file

@opbot-xd opbot-xd Aug 28, 2026

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.

Hi @regulartim, I hope its fine to add exception for this rule, or should we add notice at top of all files like we have in some files of Greedybear?

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.

I checked if we add this line at the very first line of each file (before module docstrings or imports):

# Copyright (c) 2021 The Honeynet Project

We were able to address this rule. Total 13 files required it.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hey @opbot-xd ! For me it's fine to add an exception for this rule.

@opbot-xd
opbot-xd requested a review from regulartim August 28, 2026 05:34
@opbot-xd
opbot-xd merged commit 421c42d into develop Aug 28, 2026
7 checks passed
@opbot-xd
opbot-xd deleted the chore/bump-ruff-0.16.4 branch August 28, 2026 08:36
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.

build(deps-dev): bump ruff from 0.15.20 to 0.16.4 and resolve CPY001 lint error

2 participants