Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Changes to the spin new experience #1872

Merged
merged 2 commits into from
Oct 11, 2023

Conversation

itowlson
Copy link
Contributor

@itowlson itowlson commented Oct 10, 2023

Fixes #1525.

WIP: There are more changes requested in #1525, but I'm getting it under way immediately because the e2e tests use spin new quite a bit and I want to know where they blow up sooner rather than later. I fixed one place but I'm sure there will be others! This now has everything except the proposal to generate a blank component on spin new with no template, on which I'd like to gather more feedback before implementing. That can be a follow-up PR if necessary.

And you don't know how much I wanted to call this PR "The new new thing."

Signed-off-by: itowlson <ivan.towlson@fermyon.com>
@itowlson itowlson force-pushed the template-app-name-first branch from 93dd063 to 521eb19 Compare October 10, 2023 21:39
@itowlson itowlson marked this pull request as ready for review October 10, 2023 22:39
Copy link
Contributor

@fibonacci1729 fibonacci1729 left a comment

Choose a reason for hiding this comment

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

🎉 Excellent!

@melissaklein24 melissaklein24 linked an issue Oct 10, 2023 that may be closed by this pull request
src/commands/new.rs Outdated Show resolved Hide resolved
Signed-off-by: itowlson <ivan.towlson@fermyon.com>
@itowlson itowlson force-pushed the template-app-name-first branch from a7ae57b to d9ed79b Compare October 11, 2023 00:06
@itowlson
Copy link
Contributor Author

@melissaklein24 I unlinked #16 because this doesn't currently address that. See #1525 (comment) and #1525 (comment) for more context on what fixing #16 would involve.

Copy link
Collaborator

@rylev rylev left a comment

Choose a reason for hiding this comment

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

🎉

@itowlson itowlson merged commit 59b2125 into fermyon:main Oct 11, 2023
10 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

spin new: provide application name without providing template
3 participants