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

Only trigger publish on release.published event #572

Merged
merged 2 commits into from
Mar 5, 2024

Conversation

danielhollas
Copy link
Contributor

@danielhollas danielhollas commented Mar 1, 2024

We noticed that making a release triggered multiple different release events: created, prereleased, published. So we need to tweak the workflow file so that the workflow only run for the release.publish event.

https://github.com/aiidalab/aiidalab-widgets-base/actions?query=event%3Arelease

Per GHA docs: https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#release

EDIT: I am not sure if the fix here works, need to test on another repo. Marking as draft for now

EDIT2: Never mind, turns out I can't read the docs properly. I fixed the syntax and checked on my repo, this seems to work correctly now.

@danielhollas danielhollas requested a review from unkcpz March 1, 2024 21:48
@danielhollas danielhollas marked this pull request as ready for review March 1, 2024 22:15
@danielhollas danielhollas marked this pull request as draft March 2, 2024 03:12
@danielhollas danielhollas removed the request for review from unkcpz March 2, 2024 03:12
Copy link

codecov bot commented Mar 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.07%. Comparing base (cd2face) to head (84aae31).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #572   +/-   ##
=======================================
  Coverage   87.07%   87.07%           
=======================================
  Files          27       27           
  Lines        4649     4649           
=======================================
  Hits         4048     4048           
  Misses        601      601           
Flag Coverage Δ
python-3.10 87.07% <ø> (ø)
python-3.9 87.10% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@danielhollas danielhollas requested a review from unkcpz March 2, 2024 03:35
@danielhollas danielhollas marked this pull request as ready for review March 2, 2024 03:36
@danielhollas danielhollas changed the title Only trigger publish on release:publish event Only trigger publish on release.published event Mar 2, 2024
Copy link
Member

@unkcpz unkcpz left a comment

Choose a reason for hiding this comment

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

Looks all good, Thanks! @danielhollas
Now we have everything ready, can you write a wiki page for how to release. We can then ask Sasha to confirm with it and we can implement it to other packages as well.

@danielhollas
Copy link
Contributor Author

Yep! On my todo-list for today or Wednesday. Thanks!

@danielhollas danielhollas merged commit 8190dc0 into aiidalab:master Mar 5, 2024
11 checks passed
@danielhollas danielhollas deleted the release-event branch March 5, 2024 04:48
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.

None yet

2 participants