Skip to content

chore(docs): Remove gettext references from Enum docs #2222

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

Merged
merged 1 commit into from
Jul 25, 2025

Conversation

chazwatkins
Copy link
Contributor

@chazwatkins chazwatkins commented Jul 24, 2025

Changes

  • Remove references to gettext from Ash.Type.Enum doc examples to avoid confusion that using functions on the values for label and description will occur at runtime.

closes #2172

Contributor checklist

Leave anything that you believe does not apply unchecked.

  • I accept the AI Policy, or AI was not used in the creation of this PR.
  • Bug fixes include regression tests
  • Chores
  • Documentation changes
  • Features include unit/acceptance tests
  • Refactoring
  • Update dependencies

@chazwatkins chazwatkins force-pushed the improvement/2172 branch 3 times, most recently from cbe3bb3 to 60e7fc1 Compare July 24, 2025 03:19
@chazwatkins chazwatkins changed the title improvement: Add Ash.Type.Enum label/2, description/2, details/2 callbacks chore(enum): Remove gettext references from Ash.Type.Enum docs Jul 24, 2025
@chazwatkins chazwatkins changed the title chore(enum): Remove gettext references from Ash.Type.Enum docs improvement: Add label, description, details callbacks for Ash.Type.Enum Jul 24, 2025
@chazwatkins
Copy link
Contributor Author

Ok, this one is good to go.

@zachdaniel
Copy link
Contributor

@chazwatkins Sorry to be an ass but now that I see what you mean about this just being a new callback that anyone could define, since we don't actually use it...you were right 😢 Sorry for the whiplash, but your plan of just removing gettext from the docs is the right way.

@chazwatkins
Copy link
Contributor Author

@chazwatkins Sorry to be an ass but now that I see what you mean about this just being a new callback that anyone could define, since we don't actually use it...you were right 😢 Sorry for the whiplash, but your plan of just removing gettext from the docs is the right way.

No worries, man. Don't take it as you being an ass at all. This one didn't take long anyway.

@chazwatkins chazwatkins reopened this Jul 25, 2025
@chazwatkins chazwatkins changed the title improvement: Add label, description, details callbacks for Ash.Type.Enum chore(docs): Remove gettext references from Enum docs Jul 25, 2025
@zachdaniel zachdaniel merged commit e66487c into ash-project:main Jul 25, 2025
43 checks passed
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.

docs: using gettext in Ash.Type.Enum is misleading
2 participants