Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Move dependency management to flowkit#1659

Draft
bartolomej wants to merge 1 commit intoonflow:masteronflow/flow-cli:masterfrom
onflowser:bart/refactor-to-flowkit-depsonflowser/flow-cli:bart/refactor-to-flowkit-depsCopy head branch name to clipboard
Draft

Move dependency management to flowkit#1659
bartolomej wants to merge 1 commit intoonflow:masteronflow/flow-cli:masterfrom
onflowser:bart/refactor-to-flowkit-depsonflowser/flow-cli:bart/refactor-to-flowkit-depsCopy head branch name to clipboard

Conversation

@bartolomej
Copy link
Copy Markdown
Contributor

@bartolomej bartolomej commented Jun 23, 2024

Closes #1633

Depends on: onflow/flowkit#62

Description

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

@bartolomej bartolomej changed the title Move dependeny management to flowkit Move dependency management to flowkit Jun 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Kay-Zee Kay-Zee Awaiting requested review from Kay-Zee Kay-Zee will be requested when the pull request is marked ready for review Kay-Zee is a code owner

@janezpodhostnik janezpodhostnik Awaiting requested review from janezpodhostnik janezpodhostnik will be requested when the pull request is marked ready for review janezpodhostnik is a code owner

@chasefleming chasefleming Awaiting requested review from chasefleming chasefleming will be requested when the pull request is marked ready for review chasefleming is a code owner

@nvdtf nvdtf Awaiting requested review from nvdtf nvdtf will be requested when the pull request is marked ready for review nvdtf is a code owner

@bjartek bjartek Awaiting requested review from bjartek bjartek will be requested when the pull request is marked ready for review bjartek is a code owner

@jribbink jribbink Awaiting requested review from jribbink jribbink will be requested when the pull request is marked ready for review jribbink is a code owner

@mfbz mfbz Awaiting requested review from mfbz mfbz will be requested when the pull request is marked ready for review mfbz is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Dependency Manager: Extract core logic to flowkit package

1 participant

@bartolomej
Morty Proxy This is a proxified and sanitized view of the page, visit original site.