Skip to content

Use args_in_kwargs for basemap error raising#797

Merged
seisman merged 12 commits intoGenericMappingTools:masterfrom
willschlitzer:basemap_invalid_input
Jan 21, 2021
Merged

Use args_in_kwargs for basemap error raising#797
seisman merged 12 commits intoGenericMappingTools:masterfrom
willschlitzer:basemap_invalid_input

Conversation

@willschlitzer
Copy link
Contributor

Modify the basemap function in Figure to use args_in_kwargs() to determine if one of the required arguments is passed. This also adds a required arguments test to test_basemap.py.

@willschlitzer willschlitzer added the maintenance Boring but important stuff for the core devs label Jan 21, 2021
@willschlitzer willschlitzer added this to the 0.3.0 milestone Jan 21, 2021
@willschlitzer
Copy link
Contributor Author

willschlitzer commented Jan 21, 2021

Should the test functions that aren't using @check_figures_equal use the long-form arguments or the single-character GMT aliases?

@seisman
Copy link
Member

seisman commented Jan 21, 2021

Should the test functions that aren't using @check_figures_equal use the long-form arguments or the single-character GMT aliases?

We should use long-form arguments unless we're testing the GMT aliases.

@seisman seisman merged commit 547a6fc into GenericMappingTools:master Jan 21, 2021
@willschlitzer willschlitzer deleted the basemap_invalid_input branch January 22, 2021 19:30
sixy6e pushed a commit to sixy6e/pygmt that referenced this pull request Dec 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Boring but important stuff for the core devs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants