Skip to content
This repository was archived by the owner on Apr 16, 2026. It is now read-only.

Use full versions specs for all github actions - #450

Merged
dalito merged 1 commit into
linkml:mainfrom
vladistan:use-full-version-specs-for-all-github-actions
Sep 3, 2025
Merged

dalito merged 1 commit into
linkml:mainfrom
vladistan:use-full-version-specs-for-all-github-actions

Conversation

@vladistan

Copy link
Copy Markdown
Contributor

No description provided.

@vladistan
vladistan force-pushed the use-full-version-specs-for-all-github-actions branch from a2c23c1 to 9bb5c52 Compare September 3, 2025 01:26
@vladistan
vladistan marked this pull request as ready for review September 3, 2025 01:36
@dalito

dalito commented Sep 3, 2025

Copy link
Copy Markdown
Member

Thanks for doing this. I was about to do the same. There are still some conflict markers in the files so I cannot yet approve.

@vladistan
vladistan force-pushed the use-full-version-specs-for-all-github-actions branch from 9bb5c52 to 3b0c24f Compare September 3, 2025 15:44
@codecov

codecov Bot commented Sep 3, 2025 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.38%. Comparing base (5ef72d4) to head (3b0c24f).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #450   +/-   ##
=======================================
  Coverage   77.38%   77.38%           
=======================================
  Files          52       52           
  Lines        4479     4479           
  Branches      979      979           
=======================================
  Hits         3466     3466           
  Misses        785      785           
  Partials      228      228           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@vladistan

Copy link
Copy Markdown
Contributor Author

Thanks for doing this. I was about to do the same. There are still some conflict markers in the files so I cannot yet approve.

Ahh thanks for catching this. That's why we need pre-commit. I run it after fixing the conflicts, it also fixed up some workspace and found a bug in check-dependencies workflow.

This also explains the mystery why some checks would not run last night

@dalito
dalito merged commit 512c8cd into linkml:main Sep 3, 2025
17 checks passed
@vladistan
vladistan deleted the use-full-version-specs-for-all-github-actions branch September 3, 2025 16:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants