Skip to content

blank and empty as variable names#592

Merged
fw42 merged 1 commit intomasterfrom
method_literal
Jun 4, 2015
Merged

blank and empty as variable names#592
fw42 merged 1 commit intomasterfrom
method_literal

Conversation

@fw42
Copy link
Copy Markdown
Contributor

@fw42 fw42 commented Jun 4, 2015

Fixes #591.

Not sure if this is a good fix, feels kinda hacky, but seems to work.

@pushrax @dylanahsmith

@pushrax
Copy link
Copy Markdown
Contributor

pushrax commented Jun 4, 2015

Yeah, this is basically what I was thinking. Hopefully external tags don't rely on this.

@dylanahsmith
Copy link
Copy Markdown
Contributor

That doesn't really seem hacky.

👍

fw42 added a commit that referenced this pull request Jun 4, 2015
blank and empty as variable names
@fw42 fw42 merged commit b347fac into master Jun 4, 2015
@fw42 fw42 deleted the method_literal branch June 4, 2015 17:09
@arthurnn
Copy link
Copy Markdown
Contributor

Did we add a changelog for this?

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.

Using "blank" as a variable name

4 participants