Skip to content

Conversation

carlsonp
Copy link
Contributor

No description provided.

@t8y8
Copy link
Contributor

t8y8 commented Jul 21, 2017

@RussTheAerialist does this require a CLA?

@carlsonp looks fine to me, just need to check on the above first.

Copy link
Contributor

@t8y8 t8y8 left a comment

Choose a reason for hiding this comment

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

🚀

@graysonarts
Copy link
Contributor

@t8y8 I don't know, and @benlower left for the day so I won't be able to ask until Monday.

@carlsonp
Copy link
Contributor Author

Since it's a simple fix and I'm not quite sure what is necessary for CLA, it might be better if one of you fine folks just committed the fix?

Copy link
Contributor

@t8y8 t8y8 left a comment

Choose a reason for hiding this comment

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

I'm gonna go ahead and call this 'trivial' and approve it.

@RussTheAerialist ?

@graysonarts
Copy link
Contributor

Oh yeah, let's just merge that in, that's really simple.

@graysonarts graysonarts merged commit 4d39126 into tableau:master Feb 16, 2018
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