SG-44731 Fetch git-lfs content during Azure Pipelines checkout - #77
Draft
julien-lang wants to merge 3 commits into
Draft
SG-44731 Fetch git-lfs content during Azure Pipelines checkout#77julien-lang wants to merge 3 commits into
julien-lang wants to merge 3 commits into
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates internal Azure Pipelines templates to explicitly enable Git LFS during repository checkout so pipeline jobs operate on real LFS content instead of pointer files.
Changes:
- Add an explicit
checkout: selfstep withlfs: truenear the start of job steps in the affected templates. - Document why the explicit checkout is required (Azure Pipelines default checkout doesn’t fetch LFS objects).
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
internal/run-tests-with.yml |
Adds an explicit checkout: self step with lfs: true before subsequent test steps. |
internal/release.yml |
Adds an explicit checkout: self step with lfs: true before release steps. |
internal/code-style-validation.yml |
Adds an explicit checkout: self step with lfs: true for the documentation-build job. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
69
to
+74
| steps: | ||
| - checkout: self | ||
| displayName: Git Checkout | ||
| lfs: true | ||
| # Explicit checkout step in order to retrieve LFS entities. Otherwise, Azure | ||
| # Pipelines does not retrieve LFS |
Member
Author
There was a problem hiding this comment.
I could but... code_style_validation does not need to LFS checkout so ...
Still would not hurt. But then I should do for every job and be consistent
This was referenced Aug 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
Azure Pipelines does not retrieve Git LFS content by default on a checkout, even for repos that have LFS-tracked files. Jobs that read those files (e.g. Sphinx doc generation, App Store release/test jobs) could silently work with pointer text instead of real content.
Fix
Add an explicit
checkout: selfstep withlfs: truebefore any other step, in the three pipeline templates that checkout the repository being built:code-style-validation.yml,release.yml,run-tests-with.yml.Testing
origin/master: only these 3 files change, each gets the samecheckout: self/lfs: truestep, matching the existing indentation style of that file.