Skip to content

Configure i18n-tasks to look inside lib/ dir as well#5599

Merged
zachmargolis merged 1 commit intodayton-incorporate-telephony-gemfrom
margolis-margolis-unused-i18n-keys
Nov 9, 2021
Merged

Configure i18n-tasks to look inside lib/ dir as well#5599
zachmargolis merged 1 commit intodayton-incorporate-telephony-gemfrom
margolis-margolis-unused-i18n-keys

Conversation

@zachmargolis
Copy link
Contributor

Figured out why the i18n-tasks gem thought a bunch of keys were unused

Copy link
Contributor

@kevindayton kevindayton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zachmargolis zachmargolis merged commit fb9de69 into dayton-incorporate-telephony-gem Nov 9, 2021
@zachmargolis zachmargolis deleted the margolis-margolis-unused-i18n-keys branch November 9, 2021 19:37
kevindayton added a commit that referenced this pull request Nov 12, 2021
* Move telephony gem files to lib.

* Move telephony gem specs to spec.

* Remove telephony gem references from Gemfile.

* Add pinpoint dependencies to Gemfile.

* Migrate and normalize telephony i18n yaml files.

* Move telephony spec_helper to shared_context.

* Configure i18n-tasks to look inside lib/ dir as well (#5599)

* Try resetting Telephony.config after tests to reduce test pollution (#5600)

* Change `strip.gsub` to `squish`.

* Add test for invalid phone number.
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.

2 participants