-
Notifications
You must be signed in to change notification settings - Fork 21
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
fix: Add deprecation warning for CA #1244
fix: Add deprecation warning for CA #1244
Conversation
when the resulting list becomes empty, just completely skip the "Dedicated OCM repository types"
ocm.CommonTransportFormat instead
You're right, it's not written as requirement to remove all mentions of component archive. But I don't like to read through anyhow too large documentation and then need to let my brain filter out everything which is deprecated. @morri-son what's your take on that? Shall we still generate all the documentation around CAs or not? |
thanks - good catch! I obviously missed to search for |
@hilmarf, Commands
So far those commands are not covered by this PR, if I see it correctly, right? There may be also other commands like this, which I'm not aware of. |
A separate question is, if we remove all information about component archives, do we already have enough documentation on how to achieve the same result with CTF? |
yes, that's right - #1254 is the follow up task to be done... or we are faster with the rewrite and skip it ;-) |
I don't know. I'm not using CAs. |
as mentioned above there are some commands that only work with CAs, but personally I also never used CAs, but only CTFs and think there is enough documentation on using CTFs + component constructors. Only in case someone manually creates components step by step using resource files, he may stumble into this issue. Looking at the documentation these cases are more to explain how to build a component and its different artifacts step by step, not a use case for pipelines. |
@morri-son , please check #1244 (comment) As you see, @hilmarf suggests to remove documentation for
|
I can only share my point of view :-) I do not see a benefit in mentioning CAs anywhere in the docs anymore when we're about to discontinue support for them soon. We will keep it in the code for own purposes, e.g. debugging, but we want to shift end users to CTFs already right now. Therefore I would also remove |
My only comment is that the documentation should contain what to use instead! I know that the slack message contains some hints, I don't want to open slack when I'm reading documentation about a thing I'm about to use. I want to read it right there in the deprecation notice. |
Do you have any suggestion what the user should use instead? And how? I'm happy to add the proper text. |
I have no idea. I'm not versed enough in OCM. That's why I was missing such context. :D |
Everywhere CAs are outdated, CTFs are the appropriate replacement. So whenever having the deprecation warning, there can be sentence on how to replicate the flow with CTFs |
For most cases that's a pretty simple: |
Thank you!🤩 |
What this PR does / why we need it
we don't want to support component archives (CA) anymore
Which issue(s) this PR fixes
fixes: #1242
potential follow up: #1254