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

Loc+ strings edit #359

Merged
merged 1 commit into from
Dec 23, 2016
Merged

Loc+ strings edit #359

merged 1 commit into from
Dec 23, 2016

Conversation

jbondpdx
Copy link
Contributor

General edit of the ntp module readme, in prep for localization work and adding docs strings to code.

Of note, I've (mostly) pulled out "valid values" from descriptions, but added data types at the front of the descriptions. These types appeared to cover valid values for the most part, and having them at the front of the description will make strings import easier and let the user know up front what they are dealing with.

@jbondpdx
Copy link
Contributor Author

Oh, I forgot I did a progress commit... I can squash these if you want.

@HAIL9000
Copy link
Contributor

@jbondpdx If this is ready for review I'd say go ahead and squash that WIP commit! But if you have more work to do I'd wait and squash later

@jbondpdx
Copy link
Contributor Author

jbondpdx commented Dec 22, 2016

Nah, @HAIL9000 , no more work on it... I don't PR until it's ready for review. And I've squashed the commits (someday I won't have to look that up every time!).

@tphoney
Copy link
Contributor

tphoney commented Dec 22, 2016

@jbondpdx looks good to me, nice changes 👍

@HAIL9000 HAIL9000 merged commit 5297309 into puppetlabs:master Dec 23, 2016
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.

4 participants