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

GHA: fetch tags on Python release checkout action #244

Merged
merged 1 commit into from
Dec 13, 2023
Merged

GHA: fetch tags on Python release checkout action #244

merged 1 commit into from
Dec 13, 2023

Conversation

isinyaaa
Copy link
Contributor

@isinyaaa isinyaaa commented Dec 13, 2023

Description

Fetch tags when checking out the repo on the Python release workflow.

How Has This Been Tested?

Tested on: https://github.com/isinyaaa/model-registry/actions/runs/7197484144/job/19604711876.

Merge criteria:

  • The commits and have meaningful messages; the author will squash them after approval or will ask to merge with squash.
  • Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
  • The developer has manually tested the changes and verified that the changes work

@isinyaaa isinyaaa self-assigned this Dec 13, 2023
@isinyaaa
Copy link
Contributor Author

isinyaaa commented Dec 13, 2023

@dtrifiro do you think this should work or would you rather go with fetch-depth: 0 to get all history + tags?

FYI I'm basing myself on https://github.com/actions/checkout/blob/main/README.md?plain=1#L93.

tarilabs
tarilabs previously approved these changes Dec 13, 2023
@dtrifiro
Copy link
Contributor

I think it would be ok.

What about explicitly using:

on:
  release:
    types: [published]

and explicitly creating the release (and release notes) on github?

Signed-off-by: Isabella Basso do Amaral <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Dec 13, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (df45e90) 74.52% compared to head (d1529e7) 74.52%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #244   +/-   ##
=======================================
  Coverage   74.52%   74.52%           
=======================================
  Files          17       17           
  Lines        2296     2296           
  Branches       73       73           
=======================================
  Hits         1711     1711           
  Misses        420      420           
  Partials      165      165           

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

@isinyaaa
Copy link
Contributor Author

@isinyaaa isinyaaa merged commit 7b351c2 into opendatahub-io:main Dec 13, 2023
7 checks passed
@isinyaaa isinyaaa deleted the fix-pyrel-checkout branch December 13, 2023 20:37
dhirajsb pushed a commit to dhirajsb/model-registry that referenced this pull request Jan 20, 2025
… to 0.2.4-alpha in /clients/ui/bff (opendatahub-io#244)

Bumps [github.com/kubeflow/model-registry](https://github.com/kubeflow/model-registry) from 0.2.3-alpha to 0.2.4-alpha.
- [Release notes](https://github.com/kubeflow/model-registry/releases)
- [Commits](kubeflow/model-registry@v0.2.3-alpha...v0.2.4-alpha)

---
updated-dependencies:
- dependency-name: github.com/kubeflow/model-registry
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

4 participants