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

workflows: add scorecard and codeql #1597

Merged
merged 1 commit into from
Nov 14, 2023

Conversation

tkatila
Copy link
Contributor

@tkatila tkatila commented Nov 13, 2023

Tweak token permissions and add scorecard badge.

@tkatila
Copy link
Contributor Author

tkatila commented Nov 13, 2023

CodeQL and scorecard, and scorecard badge tested on my private project. Dependabot badge didn't function in the private project, but copied the link from a functioning project.

@codecov-commenter
Copy link

Codecov Report

Merging #1597 (e364014) into main (f9bc72f) will not change coverage.
The diff coverage is n/a.

❗ Current head e364014 differs from pull request most recent head 89f8f26. Consider uploading reports for the commit 89f8f26 to get more accurate results

@@           Coverage Diff           @@
##             main    #1597   +/-   ##
=======================================
  Coverage   51.69%   51.69%           
=======================================
  Files          42       42           
  Lines        4869     4869           
=======================================
  Hits         2517     2517           
  Misses       2206     2206           
  Partials      146      146           

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@tkatila
Copy link
Contributor Author

tkatila commented Nov 13, 2023

Hold this. I'll rewrite the structure so that it is more inline with the overall file structure.

@tkatila tkatila force-pushed the workflow/scorecard-codeql branch from 89f8f26 to ab4ca9b Compare November 14, 2023 10:22
Copy link
Contributor

@mythi mythi left a comment

Choose a reason for hiding this comment

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

was there a badge for dependabot too?

.github/workflows/lib-scorecard.yaml Outdated Show resolved Hide resolved
Also add scorecard badge and fix CI/Devel badge

Signed-off-by: Tuomas Katila <tuomas.katila@intel.com>
@tkatila tkatila force-pushed the workflow/scorecard-codeql branch from ab4ca9b to 48a96cf Compare November 14, 2023 10:52
@tkatila
Copy link
Contributor Author

tkatila commented Nov 14, 2023

was there a badge for dependabot too?

I couldn't find instructions for it. And it seems it might be broken at the moment: dependabot/dependabot-core#1912

@mythi mythi merged commit 953bcb0 into intel:main Nov 14, 2023
75 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