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

Bug: Changelog didn't have the version #324

Merged
merged 2 commits into from
Jul 8, 2021
Merged

Conversation

markmandel
Copy link
Contributor

I introduced a bug when refactoring the build pipleline, wherein there wasn't a ./version file anymore which stored the current version number, such that it could be shared across build steps.

Therefore there was no version on the changelog! This fixes that by creating a version taget for the Makefile and pushing that into a file the changelog generator can use.

Work on #313

Proof that is works again:

image

I introduced a bug when refactoring the build pipleline, wherein there
wasn't a `./version` file anymore which stored the current version
number, such that it could be shared across build steps.

Therefore there was no version on the changelog! This fixes that by
creating a `version` taget for the Makefile and pushing that into a file
the changelog generator can use.

Work on #313
@markmandel markmandel added kind/bug Something isn't working kind/release Checklist for a release labels Jul 7, 2021
@google-cla google-cla bot added the cla: yes label Jul 7, 2021
@markmandel markmandel added this to the Release milestone Jul 7, 2021
@quilkin-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: a4d42a57-63d6-46c4-8a05-e4a878332937

To build this version:

git fetch [email protected]:googleforgames/quilkin.git pull/324/head:pr_324 && git checkout pr_324
cargo build

@quilkin-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: e544f9b0-9d59-4eab-a9b7-0033ce06f167

To build this version:

git fetch [email protected]:googleforgames/quilkin.git pull/324/head:pr_324 && git checkout pr_324
cargo build

@markmandel markmandel merged commit 23f4c76 into main Jul 8, 2021
@markmandel markmandel deleted the bug/changelog-version branch July 8, 2021 17:10
XAMPPRocky pushed a commit that referenced this pull request Jul 21, 2021
I introduced a bug when refactoring the build pipleline, wherein there
wasn't a `./version` file anymore which stored the current version
number, such that it could be shared across build steps.

Therefore there was no version on the changelog! This fixes that by
creating a `version` taget for the Makefile and pushing that into a file
the changelog generator can use.

Work on #313
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes kind/bug Something isn't working kind/release Checklist for a release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants