search

flux-framework / rfc

Flux RFC project
https://flux-framework.readthedocs.io/projects/flux-rfc/
7 stars 13 forks source link

rfc21: make exception note required #386

Closed chu11 closed 1 year ago

chu11 commented 1 year ago

Problem: Currently the exception note is defined as optional. Making the exception note optional makes parsing exceptions difficult.

Solution: Make an exception note required. If one is not desired, it should be set to an empty string.

per discussion in https://github.com/flux-framework/flux-core/pull/5336

I went with the language "should" set it to empty string. Because implementation could choose something different, eg. "no note specified", etc.

chu11 commented 1 year ago

@garlick Thanks. re-pushed with that tweak. I'll hold off on setting MWP once we all agree that what is in https://github.com/flux-framework/flux-core/pull/5336 is the way we want to go.