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

purged null from Hy #889

Merged
merged 1 commit into from
Aug 12, 2015
Merged

purged null from Hy #889

merged 1 commit into from
Aug 12, 2015

Conversation

gilch
Copy link
Member

@gilch gilch commented Aug 11, 2015

This pull only removes the redundant null from Hy. It is not sufficient to close #240, but it is a step in the right direction.

Rationale: We already have None, and null was barely ever used anyway. If this breaks something, a simple (def null None) will suffice to patch it.

The fate of the other redundant keywords remains open.

we already have `nil`, and `null` was barely ever used.
@paultag
Copy link
Member

paultag commented Aug 11, 2015

Yes please. Thank you!
On Aug 11, 2015 6:30 PM, "Matthew Egan Odendahl" [email protected]
wrote:

This pull only removes the redundant null from Hy. It is not sufficient
close #240 #240, but it is a step in
the right direction.

Rationale: We already have None, and null was barely ever used anyway. If
this breaks something, a simple (def null None) will suffice to patch it.

The fate of the other redundant keywords remains open.

You can view, comment on, or merge this pull request online at:

#889
Commit Summary

  • purged null from Hy

File Changes

Patch Links:


Reply to this email directly or view it on GitHub
#889.

@zackmdavis
Copy link
Contributor

Nice branch name, too. :shipit:

@zackmdavis zackmdavis merged commit a22e2ca into hylang:master Aug 12, 2015
@gilch gilch deleted the nullify branch August 12, 2015 05:48
algernon added a commit to hylang/hydiomatic that referenced this pull request Aug 12, 2015
Although it doesn't do anything useful, because hypprint will always
print nil. Still, we should be future proof!

Related to hylang/hy#889.

Signed-off-by: Gergely Nagy <[email protected]>
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.

Remove redundant keywords
3 participants