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

fix: show only generation name in spaces:info output #3128

Merged
merged 2 commits into from
Dec 6, 2024

Conversation

k80bowman
Copy link
Contributor

Work item

In the output of the spaces:info and spaces:wait commands, the "generation" field was showing the UUID and the name of the generation, instead of just the name. This was due to a change in the API. This PR updates the output to only show the name.

Testing

  • Pull down this branch and run yarn and yarn build
  • Run ./bin/run spaces:info -s FIR-SPACE-NAME. Verify that the output shows only the name of the generation and not an object with the UUID.
  • Run ./bin/run spaces:info -s CEDAR-SPACE-NAME. Verify that the output shows only the name of the generation and not an object with the UUID
  • Run ./bin/run spaces:wait -s FIR-SPACE-NAME. Verify that the output shows only the name of the generation and not an object with the UUID.

@k80bowman k80bowman requested a review from a team as a code owner December 6, 2024 15:06
@k80bowman k80bowman force-pushed the k80/fix-spaces-info-generation branch from ef837f0 to 3db8fff Compare December 6, 2024 15:11
Copy link
Contributor

@justinwilaby justinwilaby left a comment

Choose a reason for hiding this comment

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

:shipit:

@k80bowman k80bowman merged commit 48c5b8e into prerelease/10.0.0-alpha Dec 6, 2024
8 checks passed
@k80bowman k80bowman deleted the k80/fix-spaces-info-generation branch December 6, 2024 16:06
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