Skip to content

Dependency Manager: Smartly auto-apply alias from matching account#1997

Merged
chasefleming merged 4 commits into
masterfrom
cf/simplify-dep-alias
Jul 28, 2025
Merged

Dependency Manager: Smartly auto-apply alias from matching account#1997
chasefleming merged 4 commits into
masterfrom
cf/simplify-dep-alias

Conversation

@chasefleming
Copy link
Copy Markdown
Member

@chasefleming chasefleming commented Jul 23, 2025

Closes #1987


For contributor use:

  • Targeted PR against master branch
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work
  • Code follows the standards mentioned here
  • Updated relevant documentation
  • Re-reviewed Files changed in the Github PR explorer
  • Added appropriate labels

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 23, 2025

Codecov Report

❌ Patch coverage is 40.54054% with 22 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/dependencymanager/dependencyinstaller.go 40.54% 22 Missing ⚠️

📢 Thoughts on this report? Let us know!

@chasefleming chasefleming added the Improvement Technical work without new features, refactoring, improving tests label Jul 23, 2025
@chasefleming chasefleming merged commit 3623590 into master Jul 28, 2025
5 checks passed
@chasefleming chasefleming deleted the cf/simplify-dep-alias branch July 28, 2025 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Improvement Technical work without new features, refactoring, improving tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dependency Manager: Check alias account for other contracts it might ask you about

3 participants