Use @fastly/cli in the scaffolded application #19
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
harmony7
requested review from
a team and
kpfleming
and removed request for
a team
kpfleming
approved these changes
Nov 4, 2024
| In this case, `fastly.toml` has no value for `service_id` at the time you deploy, so `fastly compute publish` will prompt | ||
| you to create a Fastly service in your account for you (afterwards saving the new service's ID to your `fastly.toml` file). | ||
| In this case, `fastly.toml` has no value for `service_id` at the time you deploy, so the Fastly CLI will prompt | ||
| you to create a Fastly service in your account for you, after which it will save the new service's ID to your `fastly.toml` file. |
There was a problem hiding this comment.
Suggested change
| you to create a Fastly service in your account for you, after which it will save the new service's ID to your `fastly.toml` file. | |
| you to create a Fastly service in your account, after which it will save the new service's ID to your `fastly.toml` file. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR makes changes to use @fastly/cli now available as an npm package:
Changes to the package.json in the scaffolded application to enable usage without a global installation of the Fastly CLI:
@fastly/clitodevDependenciesso that it's available in the scaffolded application, as a tool used to build and run or publish the application.scriptscontainsstartanddeployscripts as applicable.Updates instructions to the README as well as the command output describing how to initialize an application, as well as how to run it locally or publish it to a Fastly service, as applicable.