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

Update doctests and tests to pass a list of argument strings to Session.call_module #3178

Closed
seisman opened this issue Apr 18, 2024 · 0 comments · Fixed by #3256
Closed

Update doctests and tests to pass a list of argument strings to Session.call_module #3178

seisman opened this issue Apr 18, 2024 · 0 comments · Fixed by #3256
Labels
maintenance Boring but important stuff for the core devs
Milestone

Comments

@seisman
Copy link
Member

seisman commented Apr 18, 2024

See #3132 for context.

After PR #3139, it's recommended to pass a list of argument strings rather than a single argument string to the Session.call_module method. In PR #3132, we have refactored all module wrappers (scripts in the pygmt/src directory). We need to do similar changes for other Session.call_module calls, mostly in the pygmt/clib/session.py and pygmt/test/test_clib*.

@seisman seisman added help wanted Helping hands are appreciated maintenance Boring but important stuff for the core devs labels Apr 18, 2024
@seisman seisman added this to the 0.13.0 milestone May 17, 2024
@seisman seisman removed the help wanted Helping hands are appreciated label May 17, 2024
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 a pull request may close this issue.

1 participant