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

Fixes #8519: rudder-dev KeyError: 'master' #175

Conversation

peckpeck
Copy link
Member

@peckpeck
Copy link
Member Author

There is a little bit of refactoring : I moved get_version to redmine.py
Aside of that the main idea is to add a version_id to issue information, to be able to use it directly when cloning an issue (instead of guessing it)

@peckpeck peckpeck force-pushed the bug_8519/rudder_dev_keyerror_master branch from 838e9a7 to ea83a24 Compare June 13, 2016 15:55
@peckpeck
Copy link
Member Author

Commit modified

if bug:
new_info['tracker_id'] = Config.BUG_TACKER_ID
new_info['description'] = "There was a bug in the resolution of #" + str(info['id'])
new_info['description'] = "There was a bug in the resolution of #" + issue.name
Copy link
Member

Choose a reason for hiding this comment

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

issue id instead of name?

Copy link
Member

Choose a reason for hiding this comment

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

or maybe name gives the id of the issue ?

Copy link
Member Author

Choose a reason for hiding this comment

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

name is the id + the "i" for internal issues

@Normation-Quality-Assistant

I merge and upmerged your PR automatically -- Your faithful QA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants