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

[r2r] refactor version handling #1686

Merged
merged 4 commits into from
Mar 6, 2023
Merged

[r2r] refactor version handling #1686

merged 4 commits into from
Mar 6, 2023

Conversation

onur-ozkan
Copy link
Member

@onur-ozkan onur-ozkan commented Feb 27, 2023

  • simplified version handling
  • adopted MM_VERSION to read version in a proper way
  • authors field added
  • --version, -v, version argument now prints the mm2 version

I actually wanted to get rid of MM_VERSION completely, but I saw the required effort wasn't worth for it, it's too complicated to be removed from current pipelines, source code, etc. Because I have more important tasks, I decided to adapt it instead of refactoring it completely.

Signed-off-by: ozkanonur <work@onurozkan.dev>
mm2src/mm2_main/src/mm2.rs Outdated Show resolved Hide resolved
mm2src/mm2_main/src/mm2.rs Outdated Show resolved Hide resolved
mm2src/mm2_bin_lib/build.rs Outdated Show resolved Hide resolved
Signed-off-by: ozkanonur <work@onurozkan.dev>
@onur-ozkan onur-ozkan force-pushed the refactor-versioning branch 2 times, most recently from d8910dd to 6d41fd9 Compare March 2, 2023 10:03
shamardy
shamardy previously approved these changes Mar 2, 2023
Copy link
Collaborator

@shamardy shamardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! Only a non-blocker related to comments updating.

mm2src/mm2_bin_lib/build.rs Outdated Show resolved Hide resolved
Copy link
Collaborator

@shamardy shamardy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re-approve

@ca333 ca333 merged commit be00363 into dev Mar 6, 2023
@onur-ozkan onur-ozkan deleted the refactor-versioning branch March 6, 2023 12:22
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.

3 participants