Skip to content

Add dependency track githubaction - #99

Open
idelcano wants to merge 6 commits into
developmentfrom
feature/add_githubactions
Open

idelcano wants to merge 6 commits into
developmentfrom
feature/add_githubactions

Conversation

@idelcano

@idelcano idelcano commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

📌 References
This PR enables repository access to the master branch and the secret, allowing the Dependency-Track dependency check to run.

Issue: Closes #?
https://app.clickup.com/t/4528615/869b6mend

📝 Implementation
Added the DTRACK_API_KEY secret to the shared workflow call so that the Dependency-Track jobs are no longer skipped. From now on, every push to master will generate a BOM and upload it to Dependency Track for vulnerability analysis.

📹 Screenshots/Screen capture
N/A

🔥 Testing
After merging, check the Actions tab to confirm the Dependency-Track jobs run successfully. Then open the Dependency-Track project link from the workflow summary to review the dependency analysis. If vulnerabilities are found, triage them there and resolve or suppress as appropriate.

@github-advanced-security

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@idelcano idelcano changed the title test actions test dependency track githubaction Jun 22, 2026
@idelcano idelcano changed the title test dependency track githubaction Add dependency track githubaction Jul 20, 2026
@idelcano
idelcano requested a review from adrianq July 20, 2026 15:26
@idelcano
idelcano marked this pull request as ready for review July 20, 2026 15:30

@anagperal anagperal left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

thanks @idelcano !

Two changes to the CI config here, so this PR finishes adopting the shared workflow rather than half-adopting it.

main.yml: pull_request is no filtered by branch. That filter matches the base branch, so any PR targeting a feature branch got no automated feedback at all — no tests, no CodeQL, no dependency scan — and every analysis had to be dispatched by hand. push stays limited to master/development.

 on:
      push:
        branches: ["master", "development"]
      pull_request:
      workflow_dispatch:

Removed codeql.yml. The shared workflow already runs CodeQL as a job, so it was analysing the same commit twice.

test.yml stays — it runs on every push with no PR open, which the shared workflow does not cover.

@idelcano
idelcano requested a review from anagperal August 18, 2026 11:08
@idelcano

Copy link
Copy Markdown
Contributor Author

Hi @anagperal
I've applied the changes. It seems CodeQL is equivalent, and I've removed master/development, although you will confirm this in a technical meeting due all the branch was merged using only master/dev as github actions triggers.

@anagperal

Copy link
Copy Markdown

Hi @anagperal I've applied the changes. It seems CodeQL is equivalent, and I've removed master/development, although you will confirm this in a technical meeting due all the branch was merged using only master/dev as github actions triggers.

Thanks @idelcano ! Yes, we'll discuss in the next technical decisions meeting if we want to maintain run only PRs with base branch master/main/development or change it to run on every PR regardless of base branch, in general

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