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

[nix prep] mina_version/gen.sh: make version info generation more reselient and flexible #10569

Merged
merged 1 commit into from
May 6, 2022

Conversation

balsoft
Copy link
Contributor

@balsoft balsoft commented Mar 22, 2022

Make the version info generation more flexible (by allowing to
override various information with env variables) and reselient (by
allowing git to not be present in PATH).

@balsoft balsoft requested a review from a team as a code owner March 22, 2022 09:01
@mrmr1993 mrmr1993 added the ci-build-me Add this label to trigger a circle+buildkite build for this branch label Mar 23, 2022
@mrmr1993
Copy link
Member

@balsoft can you create a version of this against develop with the merge conflicts fiexd? It should be enough to

git checkout mina-gen-version-fixes -b mina-gen-version-fixes-develop
git merge origin/develop
# fix merge conflicts
git commit
git push origin HEAD

and then create a PR pointing at develop.

@balsoft
Copy link
Contributor Author

balsoft commented Mar 28, 2022

There's already such a PR, #10620

@mrmr1993
Copy link
Member

There's already such a PR, #10620

🤦‍♂️ Thanks, so there is!

@yorickvP yorickvP added the Tweag label Apr 11, 2022
@balsoft balsoft merged commit 5dfd6a9 into compatible May 6, 2022
@balsoft balsoft deleted the mina-gen-version-fixes branch May 6, 2022 05:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-build-me Add this label to trigger a circle+buildkite build for this branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants