-
Notifications
You must be signed in to change notification settings - Fork 125
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 gmplot hook #21
Add gmplot hook #21
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Grr why can no-one find it? Read the news readme.
The hook itself is good.
Added the Apache License to hook and |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect
What you've done is fine. It only needs to be a one-liner really. |
@Legorooj I think this is ready to go. Just waiting for your approval then we can merge... |
License info is changed back to GPL. Kindly change the copyright year as required. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@amifunny good work on this.
@amifunny also, if you're not already using a local environment with the Git CLI or another more advanced Git Client (GitHub desktop at the minimum), I'd recommend you set one up. I'd also suggest you read Git Pro if you haven't already. It's really good - I learned everything I know about Git from that book and about 10 StackOverflow questions. |
@Legorooj thanks for the suggestion. I have Git CLI on my system and learned git using for this resource https://lab.github.com/ and a lot of articles with practical use cases. I think you might be under this impression because I uploaded and edited files using web interface only. I choose it over cloning and editing, as it was a simple file addition task. |
@amifunny ah ok. |
Due to the issue faced by users of gmplot with conversion to .exe using PyInstaller, this hook was created to include data files along with gmplot. Fixes #20.