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

add a script to synchronize labels on all OCA repositories #30

Merged
merged 3 commits into from
Sep 2, 2014

Conversation

yvaucher
Copy link
Member

Here is a script to help definition of labels.

It was already run but on l10n-belgium where I have no access rights.

Here is the talk about labels: #29

@sbidoul
Copy link
Member

sbidoul commented Aug 15, 2014

Can you add l10n-luxemburg to the list of repos?

Otherwise the script worked for me. 👍

@yvaucher
Copy link
Member Author

@sbidoul thanks for the test. I added missing l10n-luxembourg

Create and modify labels on github to have same labels and same color
on all repo

To use, install PyGithub
Copy link
Member

Choose a reason for hiding this comment

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

The other scripts in the repo use github3.py why not using the same library?

@guewen
Copy link
Member

guewen commented Aug 15, 2014

👎 because it does not conform to the other scripts

@yvaucher
Copy link
Member Author

@guewen thanks for the review. Fixed the script.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) when pulling 5056cbb on yvaucher:add-set-labels into d97827f on OCA:master.

@guewen
Copy link
Member

guewen commented Aug 15, 2014

Thanks 👍

@guewen
Copy link
Member

guewen commented Aug 15, 2014

Could you add a few lines in the USAGE section of the readme?

@yvaucher
Copy link
Member Author

@guewen done

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) when pulling c1c8d39 on yvaucher:add-set-labels into d97827f on OCA:master.

@guewen
Copy link
Member

guewen commented Aug 15, 2014

Thanks

@gurneyalex
Copy link
Member

👍

@gurneyalex
Copy link
Member

@yvaucher could you rebase?

@yvaucher
Copy link
Member Author

yvaucher commented Sep 2, 2014

@gurneyalex done

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) when pulling 3eaf5ee on yvaucher:add-set-labels into 1397a26 on OCA:master.

gurneyalex added a commit that referenced this pull request Sep 2, 2014
add a script to synchronize labels on all OCA repositories
@gurneyalex gurneyalex merged commit 9383192 into OCA:master Sep 2, 2014
@yvaucher yvaucher deleted the add-set-labels branch September 3, 2014 17:43
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.

None yet

5 participants