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

Renaming base fields #125

Closed
wants to merge 3 commits into from
Closed

Renaming base fields #125

wants to merge 3 commits into from

Conversation

fao89
Copy link
Member

@fao89 fao89 commented Oct 3, 2019

_id to pulp_id
_created to pulp_created
_last_updated to pulp_last_updated
_href to pulp_href

ref #5457
https://pulp.plan.io/issues/5457

Required PR: pulp/pulpcore#317
Required PR: pulp/pulpcore-plugin#137
Required PR: pulp/pulp-smash#1220

@fao89 fao89 force-pushed the 5454and5457 branch 2 times, most recently from d35bc2f to 182828b Compare October 9, 2019 14:30
_id to pulp_id
_created to pulp_created
_last_updated to pulp_last_updated
_href to pulp_href

ref #5457
https://pulp.plan.io/issues/5457
[noissue]
@daviddavis
Copy link
Contributor

@mdellweg this can/should be merged as the field name changes in core have been merged.

Copy link
Member

@mdellweg mdellweg left a comment

Choose a reason for hiding this comment

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

Thanks for the change. But i think, you added a directory that does not belong here.

@@ -0,0 +1,339 @@
GNU GENERAL PUBLIC LICENSE
Copy link
Member

Choose a reason for hiding this comment

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

I don't think, this belongs here.

Maybe add /pip-wheel-metadata to .gitignore.

Copy link
Member Author

Choose a reason for hiding this comment

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

sorry for that, I rebased so many PRs yesterday that I forgot to add it to gitignore

Copy link
Member

Choose a reason for hiding this comment

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

No problem.

@@ -0,0 +1,24 @@
Metadata-Version: 2.1
Copy link
Member

Choose a reason for hiding this comment

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

This, too.

@@ -0,0 +1,3 @@
[pulpcore.plugin]
Copy link
Member

Choose a reason for hiding this comment

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

Just the whole directory.

@@ -0,0 +1 @@
pulp_deb
Copy link
Member

Choose a reason for hiding this comment

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

Finally...

@mdellweg
Copy link
Member

for reference: pulp/plugin_template#125

@mdellweg
Copy link
Member

Also please consider using:
pulp/plugin_template#116,
pulp/plugin_template#124 and
pulp/plugin_template#125
for rebuilding the ci files.

@mdellweg
Copy link
Member

@fabricio-aguiar I cherry-picked your changes to another branch, where i had the travis update already. Thank you!

@mdellweg mdellweg closed this Oct 10, 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
Development

Successfully merging this pull request may close these issues.

None yet

3 participants