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

ncm-*: Remove variants of yesno pseudo-boolean #749

Merged
merged 1 commit into from
Apr 12, 2016

Conversation

jrha
Copy link
Member

@jrha jrha commented Apr 11, 2016

Standardise on new type from pan/legacy

quattor/template-library-core#111 introduced a common type for the variously named versions of a string containing either yes or no, this switches the components to use the new common version rather than their own differently named types.

It does not yes address the components that define types of string with a validation rule to essentially do the same thing, but without defining an explicit type.

Turns out coding on a train isn't that easy... just left Stirling, next stop Gleneagles.

Standardise on new type from pan/legacy
@stdweird stdweird changed the title ncm-*: Remove variants of yesno psuedo-boolean ncm-*: Remove variants of yesno pseudo-boolean Apr 11, 2016
@stdweird
Copy link
Member

@jrha is this ok to merge (do you have more commits?) also, maybe create an issue to handle the remaining cleanup

@stdweird stdweird added this to the 16.4 milestone Apr 12, 2016
@jrha
Copy link
Member Author

jrha commented Apr 12, 2016

You can merge this, I probably won't have any more time to finish this until I get back, also the wireless in the Hotel in Pitlochry is getting overloaded by our meeting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants