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 option to disable lockfile generation #1281

Merged
merged 5 commits into from
Nov 15, 2023
Merged

Conversation

cd-work
Copy link
Contributor

@cd-work cd-work commented Nov 9, 2023

Closes #1094

@cd-work cd-work requested a review from a team as a code owner November 9, 2023 12:03
@cd-work cd-work requested review from maxrake and kylewillmon and removed request for maxrake November 9, 2023 12:03
Copy link
Contributor

@maxrake maxrake left a comment

Choose a reason for hiding this comment

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

First pass quick review:

  • CHANGELOG entry needed
  • Docs need to be regenerated
  • There is a deno check failure in CI
  • Add more detail to the lockfile generation page for explaining the options for opt-out

@cd-work cd-work requested a review from maxrake November 10, 2023 13:12
Copy link
Contributor

@maxrake maxrake left a comment

Choose a reason for hiding this comment

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

The code was reviewed. The doc updates and changelog entries look good. Local testing revealed an issue.

cli/src/commands/parse.rs Show resolved Hide resolved
@cd-work cd-work requested a review from maxrake November 13, 2023 16:39
maxrake
maxrake previously approved these changes Nov 13, 2023
Copy link
Contributor

@maxrake maxrake left a comment

Choose a reason for hiding this comment

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

LGTM

@cd-work cd-work requested a review from maxrake November 15, 2023 13:42
Copy link
Contributor

@maxrake maxrake left a comment

Choose a reason for hiding this comment

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

Code reviewed and tested locally. LGTM.

@cd-work cd-work merged commit 5fbf7f6 into main Nov 15, 2023
13 checks passed
@cd-work cd-work deleted the force_disable_generation branch November 15, 2023 16:17
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.

Provide ability to opt-out of lockfile generation
2 participants