Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add the command 'remote clean' #3767

Merged
merged 1 commit into from Oct 18, 2018

Conversation

Projects
None yet
4 participants
@wagdav
Copy link
Contributor

commented Oct 16, 2018

Changelog: Feature: Add conan remote clean subcommand

This PR adds a new subcommand conan remote clean to remove all remotes and package-remote associations.

For our specific use case, we want to ensure that we pull only from our own private remotes. The 'remote clean' subcommand alleviates pain points when removing pre-existing remotes (e.g., conan-center).

This feature was designed in PR #3417 which was eventually rejected. This PR proposes a simpler and better method to clean the list of remotes.

Co-authored-by: @marco-m

  • [ X] Refer to the issue that supports this Pull Request.
  • [ X] If the issue has missing info, explain the purpose/use case/pain/need that covers this Pull Request.
  • [ X] I've read the Contributing guide.
  • [ X] I've followed the PEP8 style guides for Python code.
  • [ X] I've opened another PR in the Conan docs repo to the develop branch, documenting this one.
@lasote

This comment has been minimized.

Copy link
Contributor

commented Oct 17, 2018

Hi! Thanks for your contribution. I think it makes sense and it is useful.

@lasote lasote added this to the 1.9 milestone Oct 17, 2018

Show resolved Hide resolved conans/client/remote_registry.py Outdated
Add the command 'remote clean'
This feature was designed in the discussion about following PR:

#3417

Co-authored-by: marco-m <marco.molteni@laposte.net>

@jgsogo jgsogo assigned lasote and unassigned jgsogo Oct 18, 2018

@jgsogo

jgsogo approved these changes Oct 18, 2018

@lasote lasote merged commit 23385d3 into conan-io:develop Oct 18, 2018

2 checks passed

continuous-integration/jenkins/pr-head This commit looks good
Details
license/cla Contributor License Agreement is signed.
Details

@ghost ghost removed the contributor pr label Oct 18, 2018

grisumbras pushed a commit to grisumbras/conan that referenced this pull request Dec 27, 2018

Add the command 'remote clean' (conan-io#3767)
This feature was designed in the discussion about following PR:

conan-io#3417

Co-authored-by: marco-m <marco.molteni@laposte.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.