Add default acronyms: API and CSRF

dry-rb / dry-inflector

Inflector for Ruby

https://dry-rb.org/gems/dry-inflector

MIT License

95 stars 15 forks source link

Add default acronyms: API and CSRF #35

Closed jodosha closed 3 years ago

jodosha commented 3 years ago

Enhancement

With dry-rb, Hanami, ROM using a custom inflector for Zeitwerk, we may want to expand the set of default acronyms. Those acronyms are needed to Zeitwerk to correctly require classes like CSRFToken. Zeitwerk defaults to CamelCase class names, and must be instructed about the exceptions.

Added default acronyms:

API: Application Programming Interface
CSRF: Cross Site Request Forgery

solnic commented 3 years ago

@jodosha is this still a draft? asking because it LGTM already

jodosha commented 3 years ago

@solnic I wanted to add moar acronyms in this PR. We need to decide which one should go in here.

Alternatively, we can merge this one and add more in the future, when they are needed.

jodosha commented 3 years ago

@solnic Given this is a blocker for https://github.com/hanami/controller/pull/348, I'd go for merging and releasing this PR.