Skip to content

Conversation

bowlofeggs
Copy link

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 4f86420 on rbarlow:alpha_release into 637b336 on pulp:1.0-dev.

@dkliban dkliban self-assigned this Mar 23, 2015
@dkliban
Copy link
Member

dkliban commented Mar 23, 2015

LGTM

Copy link
Contributor

Choose a reason for hiding this comment

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

This is slightly different than platform, which would be something like 1.0.0a1 I believe. I think both ways work OK though 🐨

Copy link
Author

Choose a reason for hiding this comment

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

I don't personally think it's necessary for the setup.py files to have this, but if we have a team policy that it should I can change it later. For now, I'll leave it as is since primarily the spec file will communicate its "alphaness".

@dkliban dkliban added the LGTM label Mar 23, 2015
@bowlofeggs bowlofeggs merged commit 4f86420 into pulp:1.0-dev Mar 23, 2015
@bowlofeggs bowlofeggs deleted the alpha_release branch March 23, 2015 23:50
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.

4 participants