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

Fixed group and label assignment bug in Element_Operation.__call__ #1119

Merged
merged 1 commit into from Feb 10, 2017

Conversation

@drs251
Copy link
Contributor

@drs251 drs251 commented Feb 10, 2017

Reopen of PR #1110, to address issue #1106. @philippjfr Perhaps it will work this time, if not just go ahead and make your own PR - I really need more practice with git...

@philippjfr
Copy link
Member

@philippjfr philippjfr commented Feb 10, 2017

Thanks so much for going through the effort, practice makes perfect. I'm sure it'll work this time.

Loading

@philippjfr
Copy link
Member

@philippjfr philippjfr commented Feb 10, 2017

Okay our test build issues persist but now it's on me to figure out a solution.

Loading

Copy link
Member

@philippjfr philippjfr left a comment

Looks good and tests should pass shortly, will go ahead and merge when that happens.

Loading

@philippjfr
Copy link
Member

@philippjfr philippjfr commented Feb 10, 2017

Tests passed at last. Thanks again and congrats on your first PR!

Loading

@philippjfr philippjfr merged commit 2654bd5 into holoviz:master Feb 10, 2017
3 checks passed
Loading
@drs251
Copy link
Contributor Author

@drs251 drs251 commented Feb 10, 2017

Awesome, thank you!

Loading

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

Successfully merging this pull request may close these issues.

None yet

2 participants