Skip to content

Conversation

orzechow
Copy link
Member

This adds a GitHub Action testing the latest debian package by

@orzechow orzechow requested a review from ll-nick November 12, 2024 10:38
@orzechow orzechow force-pushed the test_released_deb_package branch 2 times, most recently from 7238625 to 678561f Compare November 12, 2024 10:41
@orzechow
Copy link
Member Author

See https://github.com/KIT-MRT/util_caching/actions/runs/11795564251/job/32855536193?pr=36 for a test of this workflow.
I've removed the hook running this on each push (only runs on releases)

Copy link
Collaborator

@ll-nick ll-nick left a comment

Choose a reason for hiding this comment

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

Very nice! And I just learned about a new trigger :)

Looks good to me, no complaints on my end 🏅

@orzechow orzechow force-pushed the test_released_deb_package branch from 678561f to 8d7cf55 Compare November 12, 2024 10:47
@orzechow orzechow merged commit 4c8e810 into main Nov 12, 2024
1 check passed
@orzechow orzechow deleted the test_released_deb_package branch November 12, 2024 10:48
@orzechow orzechow restored the test_released_deb_package branch November 12, 2024 10:59
@orzechow orzechow deleted the test_released_deb_package branch November 12, 2024 10:59
@orzechow
Copy link
Member Author

orzechow commented Nov 12, 2024

That of course didn't run… 🤦

Also, it would be better to actually use the ${{ github.event.release.tag_name }} explicitly, instead of assuming latest

New PR will follow 🙈
Edit → #37

@orzechow orzechow mentioned this pull request Nov 12, 2024
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.

2 participants