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

Development status #4

Closed
betaboon opened this issue May 19, 2024 · 17 comments
Closed

Development status #4

betaboon opened this issue May 19, 2024 · 17 comments
Assignees

Comments

@betaboon
Copy link
Contributor

Hello @ihavespoons,

First off, thanks for setting this up.

I wanted to ask if you're actively maintaining this and whether you're open for PRs.

I've got several ideas on how to make this ruleset more feature-complete and more convenient to use.

Looking forward to hearing back from you.

all the best.

@ihavespoons
Copy link
Owner

Hey @betaboon,

Cheers for reaching out,

I am certainly actively maintaining this although in the past couple of months I haven't actively been adding features due to some role changes.

I'd love to see and hear your ideas and I am happy to assist in anyway I can from review, to writing to whatever you'd like

Thanks

@ihavespoons ihavespoons self-assigned this May 19, 2024
@betaboon
Copy link
Contributor Author

betaboon commented May 20, 2024

Here's a list of things I'm considering/playing with atm.

Looking forward to hearing your opinions :)

this is my WIP-branch: main...betaboon:rules_syft:feat-grype

maintenance

convenience

features

  • introduce grype-toolchain
    • currently working on this
  • introduce grype.database, grype_report and grype_test
  • rename syft_generate_sbom to syft_sbom
  • rename the type argument of syft_generate_sbom to format
    • having progressed on the grype-integration, i would actually prefer having the rule generate all output-formats and then exposing the syft-json format via a provider, so that a grype_report can rely on it as input

@betaboon
Copy link
Contributor Author

betaboon commented May 23, 2024

@betaboon
Copy link
Contributor Author

any news? :)

@ihavespoons
Copy link
Owner

Sorry @betaboon been crook the last few days. Running the workflows on that PR now

@betaboon
Copy link
Contributor Author

no worries.

any opinion on renaming syft_generate_sbom to syft_sbom?
my reasoning for it is, that the majority of rules is named as a noun.

@ihavespoons
Copy link
Owner

Don't have an issue with rename but lets keep both with a notice in docs that syft_generate_sbom will be deprecated in the future.

@betaboon
Copy link
Contributor Author

sounds good

@betaboon
Copy link
Contributor Author

done in the last commit on my PR

@ihavespoons
Copy link
Owner

Awesome appreciate it mate, just pushing the last stuff through to BCR now. Then I will do a final review for this grype change.

@ihavespoons
Copy link
Owner

Out of curiosity what got you onto this ruleset @betaboon? It felt pretty niche when I originally created it hahaha

@betaboon
Copy link
Contributor Author

quite a while ago i evaluated https://github.com/hxtk/rules_anchore, tried to get a PR in, barely any response.

now ~18 month later i had some time on my hands and was about to do what you did here :)

@ihavespoons
Copy link
Owner

Hahah awesome, you had exactly the same response I did! pretty sure my issues were deleted and all. Well I am glad to have you here. Definitely planned a lot more for this module but life and health got in the way. I want to see if we can get our hands on the rules_anchore module name at some point

@betaboon
Copy link
Contributor Author

i think aside from the grype_updater rule (which imho isn't really compatible with bzlmod) this ruleset has reached feature parity with my PR.

so maybe they're willing to deprecate in favor of this, and donate the name :)

@betaboon
Copy link
Contributor Author

thanks for merging.

could you release on BCR when you find the time?

@ihavespoons
Copy link
Owner

Hey @betaboon sorry planning to fix up some issues today then get the merge in.

@betaboon
Copy link
Contributor Author

heya. i just sent another PR to update tool-versions.
I'll close this issue, as everything discussed in here is done :)

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

No branches or pull requests

2 participants