-
Notifications
You must be signed in to change notification settings - Fork 20
Conversation
Codecov Report
@@ Coverage Diff @@
## master #51 +/- ##
=======================================
Coverage 42.33% 42.33%
=======================================
Files 22 22
Lines 711 711
=======================================
Hits 301 301
Misses 410 410
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good-- I didn't approve just because it's WIP. I'd be happy to merge as-is.
@@ -34,7 +34,7 @@ serializer needs to add that field as well. | |||
) | |||
|
|||
class Meta: | |||
fields = ContentSerializer.Meta.fields + ('relative_path',) | |||
fields = SingleArtifactContentSerializer.Meta.fields + ('relative_path',) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice catch
@asmaco I updated the PR to the latest additions to the stages API and removed the WIP status. |
87ceba4
to
db59ed6
Compare
@@ -33,6 +33,10 @@ working directory setup, and database cleanup after encountering failures. | |||
Synchronizing | |||
------------- | |||
|
|||
.. tip:: | |||
|
|||
Please consider using the high level :ref:`stages-docs` for actual implementations. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this-- could you remove a similar section above?
https://github.com/pulp/pulpcore-plugin/pull/51/files#diff-6419377e4fd81d7840183ac963bd1047L6
pulpcore/plugin/stages/api.py
Outdated
""" | ||
This method drains items from the last queue and drops them. | ||
|
||
Especially it does not try to put items into the nonexistent next queue. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Optional nitpick: s/Especially/Importantly/
@@ -91,7 +91,8 @@ class RemoveDuplicates(Stage): | |||
""" | |||
Stage allows plugins to remove content that would break repository uniqueness constraints. | |||
|
|||
This stage is expected to be added by the DeclarativeVersion. See that class for example usage. | |||
This stage is expected to be added by the | |||
:class:`~pulpcore.plugin.stages.DeclarativeVersion`. See that class for example usage. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd like to hear from @jortel that the second use of ResolveFuture is unintended. Then
1c9b94d
to
aa2f44d
Compare
@jortel, we aren't removing the second RemoveFuture anymore, so merging now. |
[noissue]