Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix(Makefile): add version validation checks#44

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

ThomasK33
Copy link
Member

@ThomasK33ThomasK33 commentedFeb 12, 2025
edited
Loading

Adds validation for version variables in Makefile

Ensures CURRENT_PROJECT_VERSION and MARKETING_VERSION are not empty when building the project, failing the build process with an error message if either variable cannot be determined from git tags.

Change-Id: I83265a11182df6d28f2ef3a5f5454adba27c95ea
Signed-off-by: Thomas Kosiewskitk@coder.com

@ThomasK33Graphite App
Copy link
MemberAuthor

This stack of pull requests is managed byGraphite. Learn more aboutstacking.

@ThomasK33ThomasK33 changed the titlefix(Makefile): add missing dollar sign and add assert for non empty version stringserFeb 12, 2025
@ThomasK33ThomasK33 changed the titleerfix(Makefile): add version validation checksFeb 12, 2025
@ThomasK33ThomasK33 self-assigned thisFeb 12, 2025
@ThomasK33ThomasK33 marked this pull request as ready for reviewFebruary 12, 2025 15:03
@ThomasK33ThomasK33force-pushed the02-12-fix_makefile_add_missing_dollar_sign_and_add_assert_for_non_empty_version_strings branch 2 times, most recently from2c83e83 to1af2771CompareFebruary 12, 2025 15:19
…ersion stringsChange-Id: I83265a11182df6d28f2ef3a5f5454adba27c95eaSigned-off-by: Thomas Kosiewski <tk@coder.com>
@ThomasK33ThomasK33force-pushed the02-12-fix_makefile_add_missing_dollar_sign_and_add_assert_for_non_empty_version_strings branch from1af2771 to9370b0aCompareFebruary 12, 2025 15:27
@ThomasK33ThomasK33 merged commitd2ed4a9 intomainFeb 12, 2025
4 checks passed
@ThomasK33ThomasK33 deleted the 02-12-fix_makefile_add_missing_dollar_sign_and_add_assert_for_non_empty_version_strings branchFebruary 12, 2025 15:38
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@ethanndicksonethanndicksonethanndickson approved these changes

Assignees

@ThomasK33ThomasK33

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@ThomasK33@ethanndickson

[8]ページ先頭

©2009-2025 Movatter.jp