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

refactor: Remove _MODE. #2892

Merged
merged 3 commits into from
Jun 6, 2024
Merged

refactor: Remove _MODE. #2892

merged 3 commits into from
Jun 6, 2024

Conversation

prmukherj
Copy link
Collaborator

@prmukherj prmukherj commented Jun 3, 2024

MESHING_MODE -> MESHING
PURE_MESHING_MODE -> PURE_MESHING

@seanpearsonuk
Copy link
Collaborator

We ought to document this change prominently if the old version don't work any more.

@prmukherj
Copy link
Collaborator Author

We ought to document this change prominently if the old version don't work any more.

Is it a better idea to support backward compatibility?

@prmukherj prmukherj enabled auto-merge (squash) June 6, 2024 04:09
@prmukherj prmukherj merged commit 79947a6 into main Jun 6, 2024
28 checks passed
@prmukherj prmukherj deleted the maint/refactor_fluent_mode_enum branch June 6, 2024 17:16
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.

Naming of FluentMode enum members is inconsistent
4 participants