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

Logic for upstream version sync added #54

Merged
merged 8 commits into from
Dec 15, 2023

Conversation

kz6fittycent
Copy link
Contributor

Due to the way Jenkins releases their LTS vs Latest, it caused an issue with the latest snap build and release to candidate, where the version built, albeit Jenkins' latest LTS, was not the same branch we've been building against. I am hopeful that this change will prevent it in the future. Otherwise, we can just set a hard number in the file (snap a line on a specific version) and anything greater than that would be built, and lower version numbers wouldn't - I hope.

This is working in local testing but I do not have the same actions set up for my repos, yet, so I have to rely on this to perform the test.

This is the default, but I am concerned that the release tags will still cause an issue with the workflows triggering another build against the LTS.
Used some bash to make sure that the upstream version is always newer than the last version in the snapcraft.yaml, since the LTS version is always behind.
matches `stable` release for the snap
snap/snapcraft.yaml Outdated Show resolved Hide resolved
kz6fittycent and others added 2 commits December 15, 2023 09:52
Co-authored-by: Jon Seager <jon@sgrs.uk>
Co-authored-by: Jon Seager <jon@sgrs.uk>
@kz6fittycent
Copy link
Contributor Author

@jnsgruk I'll take a look at using jq in the future. I'd need to rewrite several lines in the check to get that working as intended. I've got a lot going on today, so maybe I can get to a redo in the future.

@jnsgruk jnsgruk merged commit 36ec41c into snapcrafters:candidate Dec 15, 2023
1 check passed
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