-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Cleanup after #13093 #13240
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
Cleanup after #13093 #13240
Conversation
@geropl added hold because if we remove it from that list I don't think we'll ever build the package. Is this urgent or something we can chat about Monday? |
/unhold @mads-hartmann Package was removed here, I'm just removing the reference so |
@geropl the package wasn't removed it was turned into a dynamic package |
@mads-hartmann Ah-a! Sorry for not reading up on that context. 🙇 But also, hearing about "dynamic packages" the first time 😕 |
@geropl Let's chat Monday (I have one foot out the door |
Description
Removes reference to
install/preview:docker
build targetRelated Issue(s)
Context: previous PR removing said target: #13093
How to test
Release Notes
Documentation
Werft options:
If enabled this will build
install/preview
Valid options are
all
,workspace
,webapp
,ide