-
-
Notifications
You must be signed in to change notification settings - Fork 497
Add the webapp pack #1010
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
Add the webapp pack #1010
Conversation
Thanks for the PR 😍 How to test these changes in your application
Diff between recipe versionsIn order to help with the review stage, I'm in charge of computing the diff between the various versions of patched recipes. |
62099c1
to
1307d44
Compare
1307d44
to
a944852
Compare
I agree on this idea ... but the
I guess it refers to "web apps", not "apps in general" or specific apps such as "CLI apps", "microservices apps", etc. That's why I think we should make it more clear that this is recommended for "web apps". Maybe we could think of other name? Random idea: |
I would not use |
Would |
The only downside is that |
For reference purposes, Spring Framework has many kinds of starter project templates and they have one called "Spring Boot" which:
Also, we'd need to know if this concept of "project templates" is going to expand. "CLI apps" probably not needed because of what Fabien said above, but similar to what Apple's Xcode shows in "Create New Project" window: We may want (in the future) to create a "cloud-first app", a "SPA app", etc. |
We don't have a concept of project templates as we promote composition instead. So, a project is a blend of packages and packs that you can mix the way you want. |
a944852
to
a4dd3cf
Compare
PR updated accordingly. |
a4dd3cf
to
5408b4a
Compare
I'd like to deprecate the
website-skeleton
in favor of a newwebapp
pack.The recommended pack is opinionated as it promotes the following:
We might add additional opinionated choices over time.
The idea is to get something that works out of the box for most applications.