Skip to content

Add cli arg to skip dependencies#1819

Merged
casey merged 11 commits intocasey:masterfrom
ngharrington:skip_dependencies
Jan 9, 2024
Merged

Add cli arg to skip dependencies#1819
casey merged 11 commits intocasey:masterfrom
ngharrington:skip_dependencies

Conversation

@ngharrington
Copy link
Contributor

@ngharrington ngharrington commented Jan 5, 2024

New PR to address #1314

@ngharrington ngharrington changed the title Add cli arg to skip dependencies WIP: Add cli arg to skip dependencies Jan 5, 2024
@ngharrington ngharrington changed the title WIP: Add cli arg to skip dependencies Add cli arg to skip dependencies Jan 5, 2024
@casey casey enabled auto-merge (squash) January 9, 2024 08:37
@casey casey merged commit b66cce0 into casey:master Jan 9, 2024
@casey
Copy link
Owner

casey commented Jan 9, 2024

Nice! I refactored it a little:

  • Changed the name to --no-deps, since that seems more natural
  • no_depsendencies is on the RecipeContext, so I got it from there, instead of passing it separatelly

@nogweii nogweii mentioned this pull request Feb 23, 2024
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