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

Templates for issue types and update PR template #22

Merged
merged 1 commit into from
Aug 17, 2018
Merged

Conversation

leouieda
Copy link
Member

Github allows using different templates for different types of issues. I
created one for bug reports and one for feature requests. We can add as
many as we want later ("questions" for example).

Update the PR template with a checklist of reminders to make sure the
code is following the general style, has documentation, tests, etc.

Github allows using different templates for different types of issues. I
created one for bug reports and one for feature requests. We can add as
many as we want later ("questions" for example).

Update the PR template with a checklist of reminders to make sure the
code is following the general style, has documentation, tests, etc.
@leouieda
Copy link
Member Author

@GenericMappingTools/core what do you think?

Copy link
Member

@PaulWessel PaulWessel left a comment

Choose a reason for hiding this comment

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

Yes, these are good!

@PaulWessel
Copy link
Member

Good to go

@PaulWessel PaulWessel merged commit 49baf49 into master Aug 17, 2018
@PaulWessel PaulWessel deleted the templates branch August 17, 2018 23:49
obaney referenced this pull request in obaney/gmt Aug 18, 2021
Implement a low-level hook to `GMT_Create_Data` (used to create 
a GMT dataset that can be populated with numpy array data).

Introduces the new module `gmt/clib/io.py`. The function is not 
accessible from the public API (i.e., `LibGMT`). They will be serve as 
a basis for a high-level virtual file API.

Lays the foundation for #22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants