Skip to content
This repository has been archived by the owner on Oct 28, 2019. It is now read-only.

Remove duplicates does not work properly if there is multiple content… #81

Closed
wants to merge 1 commit into from

Conversation

ipanova
Copy link
Member

@ipanova ipanova commented Mar 29, 2019

… types in the repo.

closes #4604
https://pulp.plan.io/issues/4604

Please be sure you have read our documentation on creating PRs:
https://docs.pulpproject.org/en/3.0/nightly/contributing/pull-request-walkthrough.html

ipanova added a commit to ipanova/pulp_docker that referenced this pull request Mar 29, 2019
ipanova added a commit to ipanova/pulp_docker that referenced this pull request Mar 29, 2019
ipanova added a commit to ipanova/pulp_docker that referenced this pull request Mar 29, 2019
ipanova added a commit to ipanova/pulp_docker that referenced this pull request Apr 1, 2019
ipanova added a commit to ipanova/pulp_docker that referenced this pull request Apr 1, 2019
@dralley
Copy link
Contributor

dralley commented Apr 1, 2019

Can this PR be closed now that there is a second one?

@bmbouter
Copy link
Member

bmbouter commented Apr 1, 2019

Yes I lgtm'd the other one which also has Travis passing. I'm closing this one.

@bmbouter bmbouter closed this Apr 1, 2019
@codecov
Copy link

codecov bot commented Apr 1, 2019

Codecov Report

Merging #81 into master will decrease coverage by 0.05%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
- Coverage   42.93%   42.87%   -0.06%     
==========================================
  Files          22       22              
  Lines         722      723       +1     
==========================================
  Hits          310      310              
- Misses        412      413       +1
Impacted Files Coverage Δ
pulpcore/plugin/stages/association_stages.py 22.64% <0%> (-0.44%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 933a793...d543258. Read the comment docs.

@ipanova
Copy link
Member Author

ipanova commented Apr 1, 2019

@bmbouter @dralley yes sorry i was waiting on the travis to pass on the other pr to close this one

@bmbouter
Copy link
Member

bmbouter commented Apr 1, 2019

It's all good I was doing the same thing when Travis was being strange. Thank you for the PRs! 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants