Skip to content

Comments

Added capitalization rules to Constants class#90

Merged
derek73 merged 2 commits intoderek73:masterfrom
jcp:capitalization
Dec 12, 2019
Merged

Added capitalization rules to Constants class#90
derek73 merged 2 commits intoderek73:masterfrom
jcp:capitalization

Conversation

@jcp
Copy link
Contributor

@jcp jcp commented Jul 20, 2019

These changes let you set capitalization rules that are applied during instantiation.

Here are a few quick notes:

  • I added the handle_capitalization method to keep inline with how you've setup post processing within the parser.
  • The capitalize method, when used with force=True, takes precedence over the force_mixed_case_capitalization setting. This ensures you get what you expect.

jcp added 2 commits July 20, 2019 12:22
This also fixes a few minor spelling and spacing errors and ignores Pipenv files
These changes let users apply capitalization rules to all `HumanName` instances.
@jcp jcp force-pushed the capitalization branch from e3b7293 to f4bbf42 Compare July 20, 2019 20:18
@derek73 derek73 merged commit feb5832 into derek73:master Dec 12, 2019
@derek73 derek73 added this to the v1.0.5 milestone Dec 12, 2019
@derek73
Copy link
Owner

derek73 commented Dec 12, 2019

Thanks for this really useful addition to the parser. I definitely think there are others who have wanted this so I know it will be appreciated. Sorry it took me so long to get it merged. It's included in v1.0.5 published today.

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