Skip to content

chore: a bit of infrastructure housekeeping#957

Closed
mangodxd wants to merge 1 commit intoCycloneDX:mainfrom
mangodxd:cleanup/secure-400417796
Closed

chore: a bit of infrastructure housekeeping#957
mangodxd wants to merge 1 commit intoCycloneDX:mainfrom
mangodxd:cleanup/secure-400417796

Conversation

@mangodxd
Copy link
Copy Markdown

@mangodxd mangodxd commented Apr 7, 2026

A bit of infrastructure housekeeping 🚀

Hey there! I was browsing through the repo and noticed some opportunities to modernize the codebase a bit.

What's inside?

Why?
Modern tooling like ruff and pyproject.toml makes the project much easier to maintain and more welcoming for new contributors. Plus, having a CI/CD safety net is always a win.

Hope this helps! Feel free to merge if it looks good, or just close it if you're not interested. Cheers!


Pulled with ❤️ by me (ofc).

@mangodxd mangodxd requested a review from a team as a code owner April 7, 2026 10:44
@codacy-production
Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

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.

i dont see why we would want this change.
we already have a securityguide in place: https://github.com/CycloneDX/cyclonedx-python-lib?tab=security-ov-file#reporting-security-issues

@jkowalleck
Copy link
Copy Markdown
Member

Hello @mangodxd , the things in tour PR's description are debatable and should be a ticket, not a merge request.
Furthermore, you dont address any of the description's topic in your PR.


Anyway - closing as spam,
just like the pull-request template - that you ignored and removed from your PR description - stated, see

<!--🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅
You can expedite processing of your PR by using this template to provide context
and additional information. Before actually opening a PR please make sure that it
does NOT fall into any of the following categories
🚫 Spam PRs (accidental or intentional) - these will result in a 30-days or even
∞ ban from interacting with the project depending on reoccurrence and severity.
🚫 Lazy typo fixing PRs - if you fix a typo in a file, your PR will only be merged
if all other typos in the same file are also fixed with the same PR
🚫 If you fail to provide any _Description_ below, your PR will be considered spam.
If you do not check the _Affirmation_ box below, your PR will not be merged.
🚫 If you do not check one of the _AI Tool Disclosure_ boxes below, your PR will
not be merged. If you used AI tools to assist you in writing code, but fail to
provide the required disclosure, your PR will not be merged.
🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅-->

justification:

  • did not address the assumptions in the PR descriptions in any proposed changes
  • did not sign the PR - as requested in contribution guidelines
  • did not follow PR template

if you think this was a mistake, please read the according sectioning in the PR template and our contribution guidelines: https://github.com/CycloneDX/cyclonedx-python-lib/blob/main/CONTRIBUTING.md

@jkowalleck jkowalleck closed this Apr 12, 2026
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