Federico Builes

Results 53 comments of Federico Builes

@pushcx Thanks for the quick feedback. The idea behind the PR was to improve the users' privacy by a tiny amount, it stops folks from entering a bunch of emails...

@PatMyron @Aaronkala Thanks for the report! Do these other event types support these fields? https://github.com/actions/dependency-review-action/blob/0e686847c07129d741636f8826e19766b940064b/src/main.ts#L21-L24 If that's the case I think we can update the code to include them. I'm...

Thanks for the insight @Aaronkala! Maybe it'd make sense to add extra optional parameters to the Action so it accepts head/base revisions. Leaving this open while considering other options (PRs...

Thanks to the amazing work by @WillDaSilva the Action should [now support event types different to "pull_request"](https://github.com/actions/dependency-review-action/pull/165). You can test this by using the `optional-refs` branch of this repo: ```yml...

Actually both PRs already contain documentation, [2.1.0](https://github.com/actions/dependency-review-action/releases/tag/v2.1.0) has been released with support for other event types! If you see something behaving funny please open a new issue.

@tspascoal I love the idea of having one check per feature, but the main Action one should never be green if one of the children checks fails. Can we overcome...

> Backward compatibility (failing the action if one of the checks fails) can be achieved by explicitly setting the parameter fail-on-violation to true (default value is false). > This allows...

I keep forgetting about auth, thanks for bringing it up! We can wait until we have the new config file format before revisiting this.

Hi @aaronosb, thanks for the report. From the screenshot it looks like you should not be getting this warning indeed, do you see the same error if you view the...

@aaronosb If this is for a public repo, can you post a link? If not, can you email me the username/organization name and repo name to my username at github.com?