Skip to content
This repository has been archived by the owner on Sep 28, 2023. It is now read-only.

Use Enum for modality names? #15

Open
JackKelly opened this issue Oct 20, 2021 · 1 comment
Open

Use Enum for modality names? #15

JackKelly opened this issue Oct 20, 2021 · 1 comment
Labels
discussion enhancement New feature or request

Comments

@JackKelly
Copy link
Member

Instead of using strings like 'NWP' to identify each modality, should we use an Enum? (Just to make sure that folks don't accidentally type the wrong modality name, which might break things in weird ways?)

@JackKelly JackKelly added discussion enhancement New feature or request labels Oct 20, 2021
@jacobbieker
Copy link
Member

Yeah, that makes sense to do it that way

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
discussion enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants