Skip to content

Comments

Upload packages on PYPI#2

Open
trygveasp wants to merge 1 commit intoACCORD-NWP:mainfrom
trygveasp:feature/automatic_build_package
Open

Upload packages on PYPI#2
trygveasp wants to merge 1 commit intoACCORD-NWP:mainfrom
trygveasp:feature/automatic_build_package

Conversation

@trygveasp
Copy link

Could also be changed to a private REPO

Anyway we would need to enable a local package repo or have a token for the pypi repo.

It would be convenient to have built packages as it is needed for other packages using grib2sqite as a dependency from pypi

@trygveasp
Copy link
Author

Can not add any reviewers @uandrae @mfroelund

@uandrae
Copy link
Contributor

uandrae commented Feb 18, 2026

Fixed!


on:
release:
types: [created]
Copy link
Contributor

Choose a reason for hiding this comment

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

Not completely sure what the "created" type actually trigger on. Will it trigger also for drafting a new release? Then maybe using published is better ?

https://docs.github.com/en/actions/reference/workflows-and-actions/events-that-trigger-workflows#release

Copy link
Author

Choose a reason for hiding this comment

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

Good point. I never draft my releases, so it works the same way. Could be changed.

Copy link
Contributor

@mfroelund mfroelund left a comment

Choose a reason for hiding this comment

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

Looks good, I just have one question/comment. Also have you tested it?

@trygveasp
Copy link
Author

Looks good, I just have one question/comment. Also have you tested it?

This is not fully tested, but it's used in https://github.com/metno/pysurfex/actions/runs/15667240243 (copied from there, and I have tested build)

However, as this PR is at the moment it will not work as we need to make a decision on where to upload the wheels (no secret in repo and I don't want to use my pypi token for this)

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