Add --base-url option to aio app init --repo#822
Add --base-url option to aio app init --repo#822obarcelonap wants to merge 2 commits intoadobe:masterfrom
--base-url option to aio app init --repo#822Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
07654a3 to
d8115cf
Compare
|
This looks good, however, I would prefer it be done without an additional flag.
I think we should similarly overload the --repo flag to support these cases. |
|
@purplecabbage thanks for your feedback. I am not sure if we can mimic npm's behaviour since the repo option allows to reference specific directories ( In regards your examples: I would not overcomplicate the implementation. This pull request seems a quick win for me to support github enterprise installation. While I agree your proposal is much nicer I believe it requires a redesign of the command specification which I am not sure if is worthy. |
|
@shazron @purplecabbage can I get your feedback based on my last comment? If still is desired to use the npm approach, I was wondering how a concrete example would look like. Is that sufficient from your perspective? |
Description
Closes adobe/aio-cli#695
Related Issue
Motivation and Context
In order to be able to use quickstarts in Github's Enterprise Server.
How Has This Been Tested?
Unit testing
Screenshots (if appropriate):
Types of changes
Checklist: