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

Introduce construction/extension required properties #76

Open
medikoo opened this issue Nov 23, 2016 · 4 comments
Open

Introduce construction/extension required properties #76

medikoo opened this issue Nov 23, 2016 · 4 comments
Assignees

Comments

@medikoo
Copy link
Owner

medikoo commented Nov 23, 2016

/cc @kamsi

It should apply to both creating new classes (extensions), and creating new instances.

Currently we have just required setting, and all it does it prevents deletion of already set properties that have required set to true.

We need a mean, through which initialization of new object (or class) will crash when some properties are not provided to constructor (still at this point I'm not sure what should be the name of property that will mark that)

@medikoo medikoo self-assigned this Nov 23, 2016
@kamsi
Copy link
Collaborator

kamsi commented Nov 23, 2016

@medikoo requiredByDefinition perhaps?

@medikoo
Copy link
Owner Author

medikoo commented Nov 23, 2016

Or betterrequiredByDefinition ;-)
I'm not sure does it say more than required (of which you can also say required by definition).

Probably constructionRequired would be nicer, but I'm not convinced by that name either

@mtuchowski
Copy link
Collaborator

mandatory ?

@medikoo
Copy link
Owner Author

medikoo commented Nov 23, 2016

@mtuchowski I had that in mind as well for a moment, but thought that it sounds just as an alias for `required, still indeed it may be good pick :)

@medikoo medikoo changed the title Introduce contruction/extension required properties Introduce construction/extension required properties Jan 12, 2017
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

No branches or pull requests

3 participants