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

Fix release workflows #56

Merged
merged 3 commits into from
Oct 29, 2023
Merged

Fix release workflows #56

merged 3 commits into from
Oct 29, 2023

Conversation

feryardiant
Copy link
Contributor

Dari #55 & #36 kita belajar bahwa file .github/release.yml hanya sebagai konfigurasi untuk secara otomatis meng-generate release note ketika kita (secara manual) membuat release. Dalam hal ini antara release note dan release adalah 2 hal yang berbeda.

Untungnya sejak versi v0.1.14 action tersebut sudah mendukung kombinasi generate release note. Dengan cukup menambahkan field generate_release_notes

Disamping itu ketika kita push tag, event release.created akan ter-trigger makanya workflow deploy akan jalan, dan malah workflow release gak jalan karna dia menunggu release.published. Untuk itu di PR ini saya balik trigger nya.

flowchart LR
    user("Engineer
    Create a tag")
    ciRelease("CI Release
    Generate and Publish Release")
    ciDeploy("CI Deploy
    Deploy to server")

    user --push--> ciRelease
    ciRelease --published-->ciDeploy
Loading

@feryardiant feryardiant requested a review from a team October 29, 2023 01:40
@feryardiant feryardiant temporarily deployed to testing October 29, 2023 01:40 — with GitHub Actions Inactive
@github-actions github-actions bot added enhancement New feature or request integration When multiple functionalities should works together deployment Deployment related things labels Oct 29, 2023
To automate generation of release note

Signed-off-by: Fery Wardiyanto <[email protected]>
@feryardiant feryardiant temporarily deployed to testing October 29, 2023 01:48 — with GitHub Actions Inactive
@feryardiant feryardiant merged commit 2229834 into main Oct 29, 2023
6 checks passed
@feryardiant feryardiant deleted the fix-release-workflows branch October 29, 2023 01:59
feryardiant added a commit that referenced this pull request Oct 29, 2023
@feryardiant feryardiant self-assigned this Oct 29, 2023
@sakukode
Copy link

Dari diagram di atas tsb, itu berarti menjalankan 1 atau 2 workflow?

@sakukode
Copy link

Pertanyaanku, misal ketika step CI Deploy itu gagal karena faktor tertentu misal, apakah akan step di CI Release nya itu bisa otomatis dicancel juga?

@feryardiant
Copy link
Contributor Author

ketika step CI Deploy itu gagal karena faktor tertentu misal, apakah akan step di CI Release nya itu bisa otomatis dicancel juga?

Karena patokanya adalah release.published

release:
types: [published]

Jadi gak ngelihat CI nya gagak atau success, tapi apakah release nya published atau gak. Jadi kalaupun CI nya gagal, kita tetep bisa publish release manual & CI Deploy tetep jalan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployment Deployment related things enhancement New feature or request integration When multiple functionalities should works together
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants