这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@danielhardej
Copy link
Contributor

@danielhardej danielhardej commented Oct 23, 2025

I was recently working with a customer who needed help with setting up the DRA to use deny-licenses along with allow-dependencies-licenses.

The problems that came up were mainly how to specify GitHub Actions, as there is virtually no guidance on how to specify the PURL for an Action. We went through a long list of wrong ways to do it, including (but not limited to):

pkg:github/org/action@v6
pkg:actions/org/action@^6.0.0
pkg:github/my-github-action

Before landing on the right way only after inferring it from the dependency graph API.

I also add an example for scoped NPM packages as a nod to #1008.

This is just an idea, and totally appriciate if there's a better place for this so feedback/suggestions are welcome, but it's important that we have it somewhere.

@danielhardej danielhardej requested a review from a team as a code owner October 23, 2025 08:41
Copilot AI review requested due to automatic review settings October 23, 2025 08:41
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the README to provide concrete examples for configuring the allowed-dependencies-licenses option in the dependency review action. The update specifically addresses confusion around PURL (Package URL) formatting for GitHub Actions and scoped NPM packages, which previously lacked clear documentation.

Key Changes:

  • Added an example line demonstrating the allowed-dependencies-licenses configuration with proper PURL formatting for scoped NPM packages, regular NPM packages, and GitHub Actions

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@danielhardej danielhardej changed the title Update README to include allowed-dependencies-licenses example Update README to include allow-dependencies-licenses example Oct 23, 2025
@dangoor dangoor merged commit 3f464ea into actions:main Nov 4, 2025
5 checks passed
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.

3 participants