-
Notifications
You must be signed in to change notification settings - Fork 18
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
cabal-audit
: init
#148
cabal-audit
: init
#148
Conversation
- move to more declarative flake setup to avoid complexity - init the hsec-cabal cabal project - move the cabal.project file to `code`
f9efdb3
to
3e093fe
Compare
It looks great, thanks for the contribution! |
I have not done anything yet 😅 |
4a93b4e
to
c7a701f
Compare
I need to check what's wrong with |
de66da5
to
4591de7
Compare
4591de7
to
b6d99a7
Compare
I wonder if I can get rid of cabal-install s own command line parser. It’s awful for this tool where you only want to pass simple arguments. So noisy |
good work fruity fren |
note to self: |
That is a very good question. I would say it's fine to propose an older version, if and only if it satisfies the solver. |
0289b64
to
6f724c9
Compare
I have no idea why the docker build is failing |
I had a fight with GitHub Actions on Saturday, try to rebase your branch, I hope it'll be fine |
@blackheaven I recently merged; I don't think this branch lack behind |
I’ll tell you when I’m ready |
cannot use the `advisories` directory usefully
will do one more iteration of cleanup and then should be good. I wanted to add an integration test but that depends on the issue I just opened. |
- remove deps from testsuite - only create tmp dir when really needed - proper toplevel exception handling - more documentation - appease hlint - format fourmolu.yaml
dfe1790
to
f7d21be
Compare
I think this is gtg now, feel free to review; I wanted to add some tests but they seem to depend on the |
I have some follow up plans, should we merge this first or should I push here? |
@MangoIV a side note: in the SRT quarterly status report I want to give you kudos for your contributions. Shall I refer to you by "MangoIV", or by some other name? |
MangoIV is good 😅 |
🥺 👉 👈 |
Hi! can I do something to make this move forward? |
Hi! Little bump on this; what can I do to make this move forward? Mind, I also have an email by Joel where he intents to hand over the hackage namespace for Thank you <3 |
@MangoIV I'd still love to beta-test this on our work codebase but the usage command does not work for me:
|
Hi @pwm, this is because there's an incompatibility between nix versions wrt flake lock files, (to be fair, they do coin it as an experimental feature); you can fix it by upgrading your nix version to something somewhat recent. Also be sure to try out the newest version which includes a couple more fixes and allows to output a semi-structured format; nix run github:mangoiv/security-advisories/mangoiv/cabal-audit-osv#cabal-audit -- Thank you for beta-testing! edit: I have upgraded the PR description with these instructions |
This PR was rejected because it is currently out of scope for the security-avisories team. Please find the work at github.com/mangoiv/cabal-audit |
@MangoIV do you know where we can read more on what is in-scope vs. out of scope? From an end user perspective a security advisory db in itself is not that useful, it needs tooling like |
@pwm This is not something I can answer. As far as I am aware this tool was part of the "future goals" of the working group when it was founded. Perhaps open an issue on this repo? |
We should have a dedicated page but we stated it in our last report:
|
the cabal audit executable
usage
Note
if this causes some error wrt a lock file incompatibility, upgrade your nix version ;)
features implemented
open features
other changes