Skip to content
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 --skip-build CLI option #1190

Merged
merged 1 commit into from
Jul 18, 2022
Merged

Conversation

j0k3r
Copy link
Member

@j0k3r j0k3r commented Jul 4, 2022

We removed it because it was generating some deprecated message and we introduced the parameter noBuild instead.
Now the CLI option is back and the parameter is kept.

Kind of revert #1007

Double fix #965

The first commit is from #1187, it'll be dropped once the PR is merged and that one is rebased.

@j0k3r j0k3r added this to the 5.8.0 milestone Jul 4, 2022
@j0k3r j0k3r requested a review from vicary July 4, 2022 08:38
tests/index.test.js Outdated Show resolved Hide resolved
@j0k3r j0k3r force-pushed the feature/re-add-no-build-cli-option branch from 6e24bc5 to e148d35 Compare July 13, 2022 12:23
@j0k3r j0k3r changed the title Re-add the --no-build CLI option Add the --skip-build CLI option Jul 13, 2022
@j0k3r
Copy link
Member Author

j0k3r commented Jul 13, 2022

Because of Serverless automatically handling --no-build option (and we should then declare only the --build option) I prefer to use --skip-build because we don't need the --build AND no-build option. It confuses me.

We removed it because it was generating some deprecated message and we introduced the parameter `noBuild` instead.
Now the CLI option is back as a new name (see below) and the parameter is kept.

Serverless add an extra CLI option prefixed with `--no-` in case we add a new CLI option (ie: `--build`). But we don't need the `--build` option, only the `--no-build` which isn't possible. So we renamed it to `--skip-build`
@j0k3r j0k3r force-pushed the feature/re-add-no-build-cli-option branch from 8128a4d to 2132777 Compare July 18, 2022 08:33
@j0k3r j0k3r requested a review from vicary July 18, 2022 08:33
@j0k3r j0k3r merged commit 49c6353 into master Jul 18, 2022
@j0k3r j0k3r mentioned this pull request Jul 20, 2022
@j0k3r j0k3r deleted the feature/re-add-no-build-cli-option branch October 21, 2024 13:45
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.

local invoke with --no-build flag(s) causes deprecation warnings
2 participants