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

Capitalize AmountMath but leave camelCase version for now #3045

Closed
5 tasks done
katelynsills opened this issue May 6, 2021 · 0 comments · Fixed by #3892
Closed
5 tasks done

Capitalize AmountMath but leave camelCase version for now #3045

katelynsills opened this issue May 6, 2021 · 0 comments · Fixed by #3892
Assignees
Labels
Epic ERTP package: ERTP

Comments

@katelynsills
Copy link
Contributor

katelynsills commented May 6, 2021

@dtribble suggested capitalizing AmountMath, since it is similar conceptually to Math. We will want to make sure that we leave the camelCase version amountMath for now, so that there are no breaking changes.

This task can be divided up into the following PRs:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Epic ERTP package: ERTP
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant