Skip to content

feat(ci): add Dependency and License checks to build workflow#377

Open
jenstroeger wants to merge 1 commit intostagingfrom
add-dependency-review-for-pr
Open

feat(ci): add Dependency and License checks to build workflow#377
jenstroeger wants to merge 1 commit intostagingfrom
add-dependency-review-for-pr

Conversation

@jenstroeger
Copy link
Copy Markdown
Owner

I stumbled upon the dependency-review-action which looked useful. Not sure if build.yaml is a good place, or better pr-change-set.yaml. What do you think, @behnazh?

@jenstroeger
Copy link
Copy Markdown
Owner Author

Closing for further discussion in issue #5.

@thbeu
Copy link
Copy Markdown
Contributor

thbeu commented Nov 23, 2022

Hm, why was it closed? I'd prefer to run the dependency-review-action on PR whenever the pyproject.toml changes.

@jenstroeger
Copy link
Copy Markdown
Owner Author

Oh… 😳

I closed it so we can discuss first where this action should run, and I thought that issue #5 is a good place for that discussion. Reopening this PR, feel free to chime in!

@jenstroeger jenstroeger reopened this Nov 23, 2022
@behnazh
Copy link
Copy Markdown
Collaborator

behnazh commented Jan 19, 2023

I stumbled upon the dependency-review-action which looked useful. Not sure if build.yaml is a good place, or better pr-change-set.yaml. What do you think, @behnazh?

Currently the dependencies might change at the last stage when pushing to main and bumping the version. So technically pr-change-set.yaml is not enough unless we want to check licenses only when dependencies are added/removed through PRs and not worry about updates through bump commit to main because the license won't change.

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