Skip to content

SG-44731 Fetch git-lfs content during Azure Pipelines checkout - #77

Draft
julien-lang wants to merge 3 commits into
masterfrom
ticket/SG-44731-use-git-lfs
Draft

SG-44731 Fetch git-lfs content during Azure Pipelines checkout#77
julien-lang wants to merge 3 commits into
masterfrom
ticket/SG-44731-use-git-lfs

Conversation

@julien-lang

@julien-lang julien-lang commented Aug 10, 2026

Copy link
Copy Markdown
Member

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: self step with lfs: true before 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

  • Reviewed the diff against origin/master: only these 3 files change, each gets the same checkout: self / lfs: true step, matching the existing indentation style of that file.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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: self step with lfs: true near 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

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

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

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