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

Rename artifact to _artifact #158

Closed
wants to merge 1 commit into from
Closed

Rename artifact to _artifact #158

wants to merge 1 commit into from

Conversation

pavelpicka
Copy link
Collaborator

@pavelpicka pavelpicka commented Jan 24, 2019

Renaming to '_artifact' to be clearer that artifact
is part of pulpcore. as continue for #4206 refactor.

re: #4282
https://pulp.plan.io/issues/4282

Required PR: pulp/pulp#3843
Required PR: pulp/pulp-smash#1164

Signed-off-by: Pavel Picka ppicka@redhat.com

pavelpicka added a commit to pavelpicka/pulp that referenced this pull request Jan 24, 2019
Renaming to '_artifact' to be clearer that artifact
is part of pulpcore. as continue for #4206 refactor.
Fix for testing.

re: #4282
https://pulp.plan.io/issues/4282

Required PR: pulp/pulp_file#158

Signed-off-by: Pavel Picka <ppicka@redhat.com>
@daviddavis
Copy link
Contributor

daviddavis commented Jan 24, 2019

Travis is not going to pass because you need the full PR url. You have Required PR: pulp/pulp#3843 and it needs to be Required PR: https://github.com/pulp/pulp/pull/3843.

Let me know when you've fixed it and I can merge.

pavelpicka added a commit to pavelpicka/pulp that referenced this pull request Jan 28, 2019
Renaming to '_artifact' to be clearer that artifact
is part of pulpcore. as continue for #4206 refactor.
Fix for testing.

re: #4282
https://pulp.plan.io/issues/4282

Required PR: pulp/pulp_file#158

Signed-off-by: Pavel Picka <ppicka@redhat.com>
pavelpicka added a commit to pavelpicka/pulp that referenced this pull request Jan 28, 2019
Renaming to '_artifact' to be clearer that artifact
is part of pulpcore. as continue for #4206 refactor.
Fix for testing.

re: #4282
https://pulp.plan.io/issues/4282

Required PR: pulp/pulp_file#158
Required PR: pulp/pulp-smash#1164

Signed-off-by: Pavel Picka <ppicka@redhat.com>
pavelpicka added a commit to pavelpicka/pulp that referenced this pull request Jan 28, 2019
Renaming to '_artifact' to be clearer that artifact
is part of pulpcore. as continue for #4206 refactor.
Fix for testing.

re: #4282
https://pulp.plan.io/issues/4282

Required PR: pulp/pulp_file#158
Required PR: pulp/pulp-smash#1164

Signed-off-by: Pavel Picka <ppicka@redhat.com>
pavelpicka added a commit to pavelpicka/pulp that referenced this pull request Jan 28, 2019
Renaming to '_artifact' to be clearer that artifact
is part of pulpcore. as continue for #4206 refactor.
Fix for testing.

re: #4282
https://pulp.plan.io/issues/4282

Required PR: pulp/pulp_file#158
Required PR: pulp/pulp-smash#1164

Signed-off-by: Pavel Picka <ppicka@redhat.com>
Renaming to '_artifact' to be clearer that artifact
is part of pulpcore. as continue for #4206 refactor.

re: #4282
https://pulp.plan.io/issues/4282

Required PR: pulp/pulp#3843
Required PR: pulp/pulp-smash#1164

Signed-off-by: Pavel Picka <ppicka@redhat.com>
@dralley
Copy link
Collaborator

dralley commented Jan 30, 2019

I'm really sorry! This PR is going to be superceded by a new story which fixes this in a different / more permanent way.

https://pulp.plan.io/issues/4366

@dralley dralley closed this Jan 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants