Skip to content

refactor(action): target ubuntu-latest explicitely when running depency-review [DT-7006]#1479

Merged
jonapich merged 1 commit intomasterfrom
feature/DT-7006-dependency-review-ubuntu-latest
Jul 16, 2024
Merged

refactor(action): target ubuntu-latest explicitely when running depency-review [DT-7006]#1479
jonapich merged 1 commit intomasterfrom
feature/DT-7006-dependency-review-ubuntu-latest

Conversation

@jonapich
Copy link
Contributor

feat: **Target ubuntu-latest directly when running dependency-review **

https://coveord.atlassian.net/browse/DT-7006

Summary of changes:

  • The dependency-review workflow now targets ubuntu-latest directly instead of using the default.

We are changing the default value of the dependency-review action to target our self-hosted runners.
Therefore, public repositories must be adjusted prior else they will not be able to run the workflow anymore.

You should adjust and merge this pull request yourself if you're OK with it.
Feel free to comment if there's an issue!

@jonapich jonapich requested review from a team and removed request for a team July 16, 2024 09:41
feat: **Target ubuntu-latest directly when running dependency-review **

https://coveord.atlassian.net/browse/DT-7006

Summary of changes:
  - The `dependency-review` workflow now targets `ubuntu-latest` directly instead of using the default.

We are changing the default value of the `dependency-review` action to target our self-hosted runners.
Therefore, public repositories must be adjusted prior else they will not be able to run the workflow anymore.

*You should adjust and merge this pull request yourself if you're OK with it.
Feel free to comment if there's an issue!*
@github-actions
Copy link
Contributor

Thanks for your contribution @jonapich !
When your pull-request is ready to be merged, check the box below to merge it

  • Merge! :shipit:

@github-actions
Copy link
Contributor

Pull Request Report

PR Title

❌ Title should follow the conventional commit spec:

(optional scope):

Example:

feat(cli): add new command

Copy link
Collaborator

@louis-bompart louis-bompart left a comment

Choose a reason for hiding this comment

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

semantic commit msg/pr title, but code lgtm

@jonapich jonapich changed the title Target ubuntu-latest explicitely when running depency-review [DT-7006] refactor(action) target ubuntu-latest explicitely when running depency-review [DT-7006] Jul 16, 2024
@jonapich jonapich changed the title refactor(action) target ubuntu-latest explicitely when running depency-review [DT-7006] refactor(action): target ubuntu-latest explicitely when running depency-review [DT-7006] Jul 16, 2024
@jonapich jonapich enabled auto-merge (squash) July 16, 2024 12:35
@jonapich jonapich merged commit 1951961 into master Jul 16, 2024
@jonapich jonapich deleted the feature/DT-7006-dependency-review-ubuntu-latest branch July 16, 2024 12:37
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.

2 participants