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

Draft: Chore/ci compare versions #101

Closed
wants to merge 2 commits into from
Closed

Conversation

inpt333
Copy link
Collaborator

@inpt333 inpt333 commented Dec 7, 2024

@borgoat just think about it (4aeda09#r150054222) and just close this PR if you don't agree. I won't be offended 😉

Just keep in mind https://github.com/appforit/appforit/actions/runs/12216881020/job/34080460035

@borgoat
Copy link
Member

borgoat commented Dec 8, 2024

I'm not sure this will do what we want it to do: this will create new release and tag when pubspec is updated, but the point of separating the release from the build number is that until the release is done on the store, we keep reusing the version number and the tag is shifted forward; until we decide to publish the app and then we "freeze" the tag by releasing on GitHub too, and then bump the version. If we do it this way we could just as well get rid of releases and tags, because they would just match the point when a commit was made, but not the actual release to the store.

@borgoat
Copy link
Member

borgoat commented Dec 8, 2024

As for the build failure, that is actually intended, and it happened because "I'm Very Stupid"™️ - I forgot to bump the pubspec version in my PR 😅 which is a good point.. we're not used to do it and I completely forgot... probably a better solution would be to have a check in the verify job, instead of waiting for things to be merged to notice. But as I said this won't really solve it; the failure will just happen in Xcode instead.

@inpt333 inpt333 closed this Dec 8, 2024
@inpt333 inpt333 deleted the chore/ci-compare-versions branch December 8, 2024 09:58
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