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

Taskname optional suffix #684

Merged

Conversation

maddenp-noaa
Copy link
Collaborator

Synopsis

Make the suffix argument to Asset.taskname() optional. In an application using uwtools, I found it convenient to call this method to get a prefix to use in custom log messages so that they look nice in line with log messages coming from the driver itself. Currently I have to do taskname(""); with this change, that call can just be .taskname().

Type

  • Enhancement (adds new functionality)

Impact

  • This is a non-breaking change (existing functionality continues to work as expected)

Checklist

  • I have added myself and any co-authors to the PR's Assignees list.
  • I have reviewed the documentation and have made any updates necessitated by this change.

@maddenp-noaa maddenp-noaa merged commit f776e48 into ufs-community:main Jan 15, 2025
2 checks passed
@maddenp-noaa maddenp-noaa deleted the taskname-optional-suffix branch January 15, 2025 21:39
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.

2 participants