Skip to content

Conversation

@mattkae
Copy link
Contributor

@mattkae mattkae commented Dec 14, 2025

What's new?

  • Mir's version strings are broken, so I am unable to inspect the version in both CMake and code in Miracle
  • Mir expects to replace @VARIABLE_NAME@ in spaces, so we should ensure that that can still happen

Caused by: e79ad71

Checklist

  • Tests added and pass
  • Adequate documentation added
  • (optional) Added Screenshots or videos

@mattkae mattkae requested a review from a team as a code owner December 14, 2025 18:28
Copy link
Contributor

@Saviq Saviq left a comment

Choose a reason for hiding this comment

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

I say we should use PROJECT_VERSION* in the templates instead.

@mattkae
Copy link
Contributor Author

mattkae commented Dec 14, 2025

I say we should use PROJECT_VERSION* in the templates instead.

Fixed!

@mattkae mattkae changed the title Versions still need to be 'set' as they are used and replace in CMake (e.g. via @MIR_VERSION@) Use PROJECT_VERSION instead of the old MIR_VERSION variables Dec 14, 2025
@mattkae mattkae requested a review from Saviq December 14, 2025 18:46
Copy link
Contributor

@Saviq Saviq left a comment

Choose a reason for hiding this comment

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

Of course 🤦, sorry.

2.25.1 incoming.

@Saviq Saviq enabled auto-merge December 14, 2025 18:48
@Saviq Saviq added this pull request to the merge queue Dec 14, 2025
Merged via the queue into main with commit 636d66b Dec 14, 2025
44 checks passed
@Saviq Saviq deleted the bugfix/versioning branch December 14, 2025 20:01
@github-actions
Copy link

TICS Quality Gate

✔️ Passed

No changed files applicable for TICS analysis quality gating.

TICS / TICS / Run TICS analysis

Saviq added a commit that referenced this pull request Dec 15, 2025
## What's new?
- Mir's version strings are broken, so I am unable to inspect the
version in both CMake and code in Miracle
- Mir expects to replace @VARIABLE_NAME@ in spaces, so we should ensure
that that can still happen

Caused by:
e79ad71

## Checklist

- [x] Tests added and pass
- [x] Adequate documentation added
- [x] (optional) Added Screenshots or videos
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