Skip to content

feat: Add flag to source prep to skip downloading source tars#207

Merged
reubeno merged 1 commit into
microsoft:mainfrom
dmcilvaney:damcilva/skip-source-on-prep-flag
May 27, 2026
Merged

feat: Add flag to source prep to skip downloading source tars#207
reubeno merged 1 commit into
microsoft:mainfrom
dmcilvaney:damcilva/skip-source-on-prep-flag

Conversation

@dmcilvaney
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings May 22, 2026 22:21
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new CLI flag to azldev component prepare-sources to avoid downloading lookaside sources during source preparation, enabling workflows that only need dist-git/spec metadata without fetching large source artifacts.

Changes:

  • Added SkipSources to PrepareSourcesOptions and exposed it as '--skip-sources' on the prepare-sources command.
  • Plumbed the flag through to source preparation by adding sources.WithSkipLookaside() when enabled.

Comment thread internal/app/azldev/cmds/component/preparesources.go Outdated
Comment thread internal/app/azldev/cmds/component/preparesources.go
Comment thread internal/app/azldev/cmds/component/preparesources.go
@dmcilvaney dmcilvaney force-pushed the damcilva/skip-source-on-prep-flag branch from fef4b91 to 3f0c267 Compare May 22, 2026 23:14
@dmcilvaney dmcilvaney changed the title feat: Add flag to source prep to skill downloading source tars feat: Add flag to source prep to skip downloading source tars May 26, 2026
@dmcilvaney dmcilvaney force-pushed the damcilva/skip-source-on-prep-flag branch from 3f0c267 to 98c0012 Compare May 26, 2026 18:54
@reubeno reubeno merged commit 6d78ec3 into microsoft:main May 27, 2026
15 checks passed
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.

3 participants