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

Feature/claudia_5427 Adding management of version in glancesync component #12

Merged
merged 4 commits into from
Sep 29, 2015

Conversation

flopezag
Copy link
Member

Reviewers

@jframos, @pratid , @chemaper

Description

  • Resolution of version management in setup.py
  • Configuration of bumpversion

Testing

No test.

@chemaper
Copy link
Contributor

LGTM

current_version = 1.0.0
commit = True
tag = True

Copy link
Member

Choose a reason for hiding this comment

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

We prefer not creating tags when bumping a new version, as this operation is performed at the very beginning of a new development sprint, just after previous version is tagged and released from master branch. So please set tag to False.

@pratid
Copy link
Member

pratid commented Sep 29, 2015

LGTM

1 similar comment
@jframos
Copy link
Contributor

jframos commented Sep 29, 2015

LGTM

jframos added a commit that referenced this pull request Sep 29, 2015
Feature/claudia_5427 Adding management of version in glancesync component
@jframos jframos merged commit 4a2f14a into develop Sep 29, 2015
@jframos jframos deleted the feature/claudia-5427 branch September 29, 2015 15: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.

None yet

4 participants