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

Remove RepositoryPublishURLSerializer#93

Merged
daviddavis merged 1 commit into
pulp:masterfrom
daviddavis:issue4678
Apr 29, 2019
Merged

Remove RepositoryPublishURLSerializer#93
daviddavis merged 1 commit into
pulp:masterfrom
daviddavis:issue4678

Conversation

@daviddavis

Copy link
Copy Markdown
Contributor

ref #4678

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

daviddavis pushed a commit to daviddavis/pulp_file that referenced this pull request Apr 26, 2019
daviddavis pushed a commit to daviddavis/pulp_file that referenced this pull request Apr 26, 2019
daviddavis pushed a commit to daviddavis/pulp_file that referenced this pull request Apr 26, 2019
asmacdo added a commit to asmacdo/pulp_python that referenced this pull request Apr 26, 2019
daviddavis pushed a commit to daviddavis/pulp_file that referenced this pull request Apr 26, 2019
daviddavis pushed a commit to daviddavis/pulp_file that referenced this pull request Apr 26, 2019
daviddavis pushed a commit to daviddavis/pulp_file that referenced this pull request Apr 26, 2019
daviddavis pushed a commit to daviddavis/pulp-certguard that referenced this pull request Apr 26, 2019
daviddavis pushed a commit to daviddavis/pulp_file that referenced this pull request Apr 26, 2019
daviddavis pushed a commit to daviddavis/pulp-certguard that referenced this pull request Apr 26, 2019
daviddavis pushed a commit to daviddavis/pulp-certguard that referenced this pull request Apr 26, 2019
daviddavis pushed a commit to daviddavis/pulp-certguard that referenced this pull request Apr 26, 2019
@codecov

codecov Bot commented Apr 26, 2019

Copy link
Copy Markdown

Codecov Report

Merging #93 into master will decrease coverage by 0.6%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #93      +/-   ##
==========================================
- Coverage   42.87%   42.27%   -0.61%     
==========================================
  Files          22       21       -1     
  Lines         723      712      -11     
==========================================
- Hits          310      301       -9     
+ Misses        413      411       -2
Impacted Files Coverage Δ
pulpcore/plugin/serializers.py 100% <ø> (ø) ⬆️
pulpcore/plugin/models/content.py

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 e281ba2...56e9afd. Read the comment docs.

@bmbouter

Copy link
Copy Markdown
Member

Removing this makes sense to me, but to confirm it's because the PublisherSerializer has already replaced it.

@daviddavis

Copy link
Copy Markdown
Contributor Author

@bmbouter you're exactly correct.

@bmbouter bmbouter left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @daviddavis ! 👍

daviddavis pushed a commit to daviddavis/pulp_file that referenced this pull request Apr 29, 2019
daviddavis pushed a commit to daviddavis/pulp_file that referenced this pull request Apr 29, 2019
asmacdo added a commit to asmacdo/pulp_python that referenced this pull request Apr 29, 2019
daviddavis pushed a commit to daviddavis/pulp-certguard that referenced this pull request Apr 29, 2019
daviddavis pushed a commit to daviddavis/pulp-certguard that referenced this pull request Apr 29, 2019
daviddavis pushed a commit to daviddavis/pulp-certguard that referenced this pull request Apr 29, 2019
daviddavis pushed a commit to daviddavis/pulp-certguard that referenced this pull request Apr 29, 2019
daviddavis pushed a commit to daviddavis/pulp_rpm that referenced this pull request Apr 29, 2019
Also remove publishers for now since they aren't being used currently.

fixes #4739
https://pulp.plan.io/issues/4739

Required PR: pulp/pulpcore-plugin#93
Required PR: https://github.com/pulp/pulpcore/pull/103/files
daviddavis pushed a commit to daviddavis/pulp-certguard that referenced this pull request Apr 29, 2019
daviddavis pushed a commit to daviddavis/pulp-certguard that referenced this pull request Apr 29, 2019
daviddavis pushed a commit to daviddavis/pulp_file that referenced this pull request Apr 29, 2019
daviddavis pushed a commit to pulp/pulp-certguard that referenced this pull request Apr 29, 2019
@daviddavis daviddavis merged commit 7658b1c into pulp:master Apr 29, 2019
daviddavis pushed a commit to daviddavis/pulp_file that referenced this pull request May 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants