Skip to content

feat: move version in gradle.properties#164

Open
inglor wants to merge 2 commits into
aerogear:masterfrom
inglor:move-version-to-properties
Open

feat: move version in gradle.properties#164
inglor wants to merge 2 commits into
aerogear:masterfrom
inglor:move-version-to-properties

Conversation

@inglor
Copy link
Copy Markdown

@inglor inglor commented Apr 7, 2023

Motivation

Being able to set the version from the command lineduring build.

What

Moving version in properties means it can be overwritten during the execution of gradle commands

Why

See motivation

How

Moving to gradle.properties are read before it reads the gradle.build file and can be overwritten from cli.

Verification Steps

Build without the -Pversion=3.0.0 and it build the one specified in the gradle.properties
Build with the -Pversion=3.0.0 and will build the version specified in cli.

Checklist:

  • Code has been tested locally by PR requester
  • Changes have been successfully verified by another team member

Progress

  • Finished task

Additional Notes

N/A

This makes it easier to set the version from the command line when we
build.

Signed-off-by: Leonidas Spyropoulos <artafinde@archlinux.org>
@alexted
Copy link
Copy Markdown

alexted commented Nov 7, 2024

@inglor Is this PR still relevant?

@inglor
Copy link
Copy Markdown
Author

inglor commented Nov 7, 2024

Yes it's still relevant for the reasons explained in the commit message.

@alexted
Copy link
Copy Markdown

alexted commented Nov 7, 2024

Could you resolve the conflicts?

@inglor
Copy link
Copy Markdown
Author

inglor commented Nov 7, 2024

Yes until next version - but maintainer needs to response otherwise the next version will also be conflicting.

@alexted
Copy link
Copy Markdown

alexted commented May 4, 2025

@pb82 сould you pay attention to this pr?

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