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

Fixes #35694 - Add foreman product version macro #9474

Merged
merged 1 commit into from Oct 31, 2022

Conversation

adamruzicka
Copy link
Contributor

No description provided.

@theforeman-bot
Copy link
Member

Issues: #35174

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

We already have many things with foreman_ prefixed. Should there be an additional macro in case you want to display the real Foreman version somewhere as well as a branded one?

@adamruzicka
Copy link
Contributor Author

I talked about it with @ShimShtein and we didn't really see a use for it right now, but of course we could keep it just in case

@ShimShtein
Copy link
Member

@adamruzicka I would suggest a deprecation message for the foreman_short_version just in case someone is using it.
@ekohl I think it will create confusion to have both. Maybe renaming it to foreman_core_short_version or something that would be more explicit.

@adamruzicka
Copy link
Contributor Author

I would suggest a deprecation message for the foreman_short_version just in case someone is using it.

Oh, I was under the impression it never made it into a release, but it already is in 3.4. Maybe we should keep it after all?

@ShimShtein
Copy link
Member

Hmmm... Let's split it to two issues: keeping the current macro vs deprecating it and adding the new macro.

I suggest adding the new macro in this PR, and later on we can discuss renaming/deprecating of the foreman_short_version.
Maybe adding a bit more explicit info to the documentation.

@adamruzicka adamruzicka changed the title Refs #35174 - Replace foreman version macro with productized variant Refs #35174 - Add foreman product version macro Oct 20, 2022
ShimShtein
ShimShtein previously approved these changes Oct 23, 2022
@ShimShtein
Copy link
Member

@ekohl any comments before I merge?

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

@adamruzicka please open a new issue. The existing one was already part of the 3.4.0 changelogs so your change wouldn't show up in the 3.5.0 changelogs (and any potential cherry picks). Other than that 👍

@adamruzicka adamruzicka changed the title Refs #35174 - Add foreman product version macro Fixes #35694 - Add foreman product version macro Oct 31, 2022
@adamruzicka
Copy link
Contributor Author

@adamruzicka please open a new issue

Done

@ekohl ekohl merged commit 57fe182 into theforeman:develop Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants