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

Merge Overview and Motivation #26

Open
lukescott opened this issue Jun 12, 2015 · 3 comments
Open

Merge Overview and Motivation #26

lukescott opened this issue Jun 12, 2015 · 3 comments
Labels

Comments

@lukescott
Copy link
Owner

Made some comments on 83291b0. Some redundancy between the two.

I'll probably get to it at some point.

@Naddiseo
Copy link
Collaborator

As I noted on the commit:
I think they should be separate since they address different issues:
Overview -> What we're trying to accomplish.
Motivation -> Why we're proposing this, and what is the current problem.

I do agree, though, that the duplicate wording in the motivation section needs to change.

@lukescott lukescott added the todo label Jun 12, 2015
@Naddiseo
Copy link
Collaborator

After reading the motivation section again, let's just get rid of that last sentence; I think all of it is covered in the overview.

@Naddiseo
Copy link
Collaborator

The order of what is mentioned in the motivation should be reversed; I think the "string" + undefined behaviour isn't the primary reason/problem for this proposal.
In order of priority I think what we're trying to address:

  • There are several slightly different / non-standardized ways of type hinting
  • They lead to some ambiguities in syntax
  • There are some strange behaviours with undefined which type hints + defaults can solve in come cases.

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

No branches or pull requests

2 participants