Skip to content

Microservices Guide: use cds up in deploy section#1925

Merged
renejeglinsky merged 1 commit into
mainfrom
rjegl/micro-cdsup
Aug 13, 2025
Merged

Microservices Guide: use cds up in deploy section#1925
renejeglinsky merged 1 commit into
mainfrom
rjegl/micro-cdsup

Conversation

@renejeglinsky
Copy link
Copy Markdown
Contributor

No description provided.

@renejeglinsky renejeglinsky requested a review from Akatuoro June 18, 2025 08:58
@renejeglinsky
Copy link
Copy Markdown
Contributor Author

@Akatuoro Can you please review? Thanks :)

Copy link
Copy Markdown
Contributor

@Akatuoro Akatuoro left a comment

Choose a reason for hiding this comment

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

Yes, cds up makes this a lot easier.
Maybe we could keep a note about the cf undeploy as this is not straight-forward and you'd typically want to do this with a sample app at some point. Either here or in the cds up section

@renejeglinsky
Copy link
Copy Markdown
Contributor Author

How about adding this script then to our samples? I never heard a voice complaining about missing undeploy docs, but I do see the need to undeploy things when you're playing around with samples. So why not put it into the samples? :)

@Akatuoro
Copy link
Copy Markdown
Contributor

How about adding this script then to our samples? I never heard a voice complaining about missing undeploy docs, but I do see the need to undeploy things when you're playing around with samples. So why not put it into the samples? :)

👍

It's already in the capire/samples package.json if that's what you mean.

The question may come up more once people are calling cds up instead of cf deploy ;) but we can also keep it out of the docs for now.

@renejeglinsky renejeglinsky merged commit a2e2f62 into main Aug 13, 2025
4 checks passed
@renejeglinsky renejeglinsky deleted the rjegl/micro-cdsup branch August 13, 2025 10:55
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.

2 participants