Skip to content

Add flow deps discover command#1761

Merged
jribbink merged 3 commits into
masterfrom
jribbink/discover-deps
Sep 23, 2024
Merged

Add flow deps discover command#1761
jribbink merged 3 commits into
masterfrom
jribbink/discover-deps

Conversation

@jribbink

@jribbink jribbink commented Sep 20, 2024

Copy link
Copy Markdown
Contributor

Closes #1754

Description

Adds flow deps discover command to allow users to interactively install Core Contracts.


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

codecov-commenter commented Sep 23, 2024

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 2.27273% with 43 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/dependencymanager/discover.go 0.00% 42 Missing ⚠️
internal/super/setup.go 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@chasefleming

Copy link
Copy Markdown
Member

@jribbink Dont forget to update the docs for these new commands please.

@jribbink jribbink merged commit f99a061 into master Sep 23, 2024
@jribbink jribbink deleted the jribbink/discover-deps branch September 23, 2024 21:44
@jribbink jribbink added Improvement Technical work without new features, refactoring, improving tests Feature A new user feature or a new package API and removed Improvement Technical work without new features, refactoring, improving tests labels Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature A new user feature or a new package API

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add ability to list core contracts in Dependency Manager

3 participants