Skip to content

[INTERNAL] Add message for v3 deprecation#1184

Merged
d3xter666 merged 3 commits intov3from
v3-deprecation-message
Dec 17, 2025
Merged

[INTERNAL] Add message for v3 deprecation#1184
d3xter666 merged 3 commits intov3from
v3-deprecation-message

Conversation

@d3xter666
Copy link
Copy Markdown
Contributor

JIRA: CPOUI5FOUNDATION-875

@coveralls
Copy link
Copy Markdown

coveralls commented Dec 17, 2025

Coverage Status

coverage: 94.651%. remained the same
when pulling bf110ff on v3-deprecation-message
into 4b753d3 on v3.

@d3xter666 d3xter666 requested a review from a team December 17, 2025 12:21
@d3xter666 d3xter666 force-pushed the v3-deprecation-message branch from d2d06b3 to 35d1541 Compare December 17, 2025 13:19
Comment thread README.md Outdated
>
> [UI5 CLI v2](https://ui5.github.io/cli/v2) and [UI5 CLI v3](https://ui5.github.io/cli/v3) have been deprecated 🚫
>
> Please migrate your projects to [UI5 CLI v4](https://ui5.github.io/cli/stable/updates/migrate-v4/)!
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What about just linking to the stable version (like here: UI5/cli#1258)?
This removes the need to update this old branch once again when v5 will become the latest stable version.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is how it's been decided in the backlog. I don't mind doing this and we can have a message like:
Please migrate your projects to the latest stable version.
But we sill still need to add the deprecated versions, like we do for v2 and v3 currently and adjustment will still be needed.

Any idea on how to mitigate this, so we can have a generic message for this whole section?

Copy link
Copy Markdown
Member

@matz3 matz3 Dec 17, 2025

Choose a reason for hiding this comment

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

We could just add the same message as in the documentation banner:

⚠️ UI5 CLI vX has been deprecated! We highly recommend migrating to [**the latest version**](https://ui5.github.io/cli/stable/).

⚠️ UI5 CLI vX has been deprecated! We highly recommend migrating to the latest version.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'll adjust the changes! Thanks!

@d3xter666 d3xter666 merged commit 639b267 into v3 Dec 17, 2025
17 checks passed
@d3xter666 d3xter666 deleted the v3-deprecation-message branch December 17, 2025 14:56
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