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: don't use fetch-tags in wheel build jobs #25996

Merged
merged 1 commit into from
Mar 11, 2024

Conversation

charris
Copy link
Member

@charris charris commented Mar 11, 2024

Backport of #25995.

This failed in combination with tag-based triggers:

More importantly, we don't need to fetch either the tags or the whole commit history, because:

(a) we don't derive the version number from the last tag, but from
parsing pyproject.toml
(b) we don't use a commit count, but only yyyymmdd.git-hash

See numpy/_build_utils/gitversion.py

Done only in the wheel build job right now, because that's where the problem is and we're in a hurry to solve it on the 2.0.x branch. However, this change should be fine everywhere.

[skip cirrus] [skip circle] [skip azp]

This failed in combination with tag-based triggers:
- issue: numpygh-25979
- cause: numpy#25981 (comment)

More importantly, we don't need to fetch either the tags or the whole
commit history, because:

(a) we don't derive the version number from the last tag, but from
    parsing pyproject.toml
(b) we don't use a commit count, but only yyyymmdd.git-hash

See `numpy/_build_utils/gitversion.py`

Done only in the wheel build job right now, because that's where the
problem is and we're in a hurry to solve it on the 2.0.x branch.
However, this change should be fine everywhere.

[skip cirrus] [skip circle] [skip azp]
@charris charris merged commit 9e5a197 into numpy:maintenance/2.0.x Mar 11, 2024
53 of 55 checks passed
@charris charris deleted the backport-25995 branch March 11, 2024 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants