Skip to content
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

ci(semgrep): Fix trunk workflow for semgrep step passing artifact run id (#30610) #30620

Merged
merged 1 commit into from
Nov 11, 2024

Conversation

spbolton
Copy link
Contributor

@spbolton spbolton commented Nov 11, 2024

Proposed Changes

  • Error in trunk workflow due to artifact run id not being passed into semgrep workflow

https://github.com/dotCMS/core/actions/runs/11782551202/job/32817867816#step:3:228

This PR fixes: #30610

@spbolton spbolton requested a review from a team as a code owner November 11, 2024 16:55
@spbolton spbolton changed the title ci: Add Semgrep to workflows (#30610) ci: Fix trunk workflow for semgrep step passing artifact run id (#30610) Nov 11, 2024
@spbolton spbolton changed the title ci: Fix trunk workflow for semgrep step passing artifact run id (#30610) ci(semgrep): Fix trunk workflow for semgrep step passing artifact run id (#30610) Nov 11, 2024
@spbolton spbolton force-pushed the issue-30610-add-semgrep-to-workflows branch 2 times, most recently from 2d59403 to b00a1ca Compare November 11, 2024 16:58
@spbolton spbolton force-pushed the issue-30610-add-semgrep-to-workflows branch from b00a1ca to 271e5cb Compare November 11, 2024 17:52
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@spbolton spbolton added this pull request to the merge queue Nov 11, 2024
Merged via the queue into main with commit b919bba Nov 11, 2024
37 checks passed
@spbolton spbolton deleted the issue-30610-add-semgrep-to-workflows branch November 11, 2024 23:35
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.

Add Semgrep to workflows
4 participants