Skip to content

SG-44526 Fetch dependencies when accessing references and templates - #159

Open
yungsiow wants to merge 1 commit into
masterfrom
ticket/sg-44526/fetch-dependencies-fix
Open

SG-44526 Fetch dependencies when accessing references and templates#159
yungsiow wants to merge 1 commit into
masterfrom
ticket/sg-44526/fetch-dependencies-fix

Conversation

@yungsiow

@yungsiow yungsiow commented Aug 4, 2026

Copy link
Copy Markdown

Fixing migration bug where dependencies were not being fetched along with a revision.
Introduced because in the migration of the fetch function, a new fetch_dependencies parameter was introduced and set to False by default. The original code did always fetched dependencies. Migrated callers of fetch function did not explicitly set this parameter to True.

@codecov

codecov Bot commented Aug 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 18.22%. Comparing base (0366181) to head (f1f050f).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #159   +/-   ##
=======================================
  Coverage   18.22%   18.22%           
=======================================
  Files          32       32           
  Lines        3090     3090           
=======================================
  Hits          563      563           
  Misses       2527     2527           
Flag Coverage Δ
Linux 18.22% <ø> (ø)
Python-3.10 18.22% <ø> (ø)
Python-3.11 18.22% <ø> (ø)
Python-3.13 18.22% <ø> (ø)
Python-3.9 18.22% <ø> (ø)
Windows 18.22% <ø> (ø)
macOS 18.22% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@carlos-villavicencio-adsk carlos-villavicencio-adsk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@chenm1adsk chenm1adsk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

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