Skip to content

remove packing ability and requirement for fastly.toml file to exist when using the js-compute-runtime cli #108

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

Merged
merged 2 commits into from
Jun 17, 2022

Conversation

JakeChampion
Copy link
Contributor

The ability to create a tarball was useful before the official fastly cli had the ability to package and deploy c@e projects. As these pieces of functionaliy are now handled by the fastly cli itself (specifically this function is used) - we can remove them from the js-compute-cli project.

@JakeChampion JakeChampion force-pushed the jake/no-fastly-toml branch from 196665b to a1e5300 Compare June 16, 2022 07:52
@JakeChampion JakeChampion force-pushed the jake/no-fastly-toml branch 3 times, most recently from 5fca086 to 9d7fda4 Compare June 16, 2022 09:17
Copy link
Contributor

@jameysharp jameysharp left a comment

Choose a reason for hiding this comment

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

Is there documentation somewhere that we need to update for the removal of the skip-pkg flag? At the least I guess this change should be mentioned in CHANGELOG.md.

Other than that, this looks good to me!

@JakeChampion JakeChampion force-pushed the jake/no-fastly-toml branch from 57bed77 to 0b6de1e Compare June 17, 2022 09:46
Jake Champion added 2 commits June 17, 2022 10:49
…mpute-runtime cli and no longer create a tarball as these pieces of functionaliy are now handled by the fastly cli itself (https://developer.fastly.com/reference/cli/compute/)
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