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

Re-release artifacts to update date #166

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ThreeDeeJay
Copy link

This would avoid confusion since the current release is from Apr 19, 2023, but the artifact is from June 1st, 2024
Also, this adds an automatic changelog since last release like this:
https://github.com/ThreeDeeJay/Open1560/releases/latest

This would avoid confusion since the current release is from Apr 19, 2023, but the artifact is from June 1st, 2024
@KcRobin9
Copy link
Collaborator

KcRobin9 commented Jun 7, 2024

Automatic changelog looks cool I think (I think Brick can best comment on the release part).
Maybe with a datestamp as well?

Something like:
cd6a609: (27-08-2021) Capture screenshots before swapping frame buffers (Brick)

@ThreeDeeJay
Copy link
Author

Hmm, it doesn't seem like the action I used supports date/timestamps
https://github.com/marvinpinto/action-automatic-releases?tab=readme-ov-file#supported-parameters
But I'm not sure it would be useful since it only lists multiple commits the first time it runs, so next commit will trigger a build that'd result in a single change being shown on releases, which would show the commit's date so it'll match the artifact's.
If builds were only triggered on major releases (like say, v#.#.# tags), then the action would list all the commits in between such major releases, but I guess we'd need to find a github action capable of displaying datestamps.
The action I used is kinda old, but it's been the only one that's worked consistently for years, but maybe there's something better nowadays.

@KcRobin9
Copy link
Collaborator

KcRobin9 commented Jun 8, 2024

Thanks for checking, but it's not a big deal, just nice to have if it was readily available

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