Skip to content

[2.9] Added missing lowercase keywords#1491

Closed
michalbundyra wants to merge 1 commit intosquizlabs:2.9from
michalbundyra:hotfix/lowercase-keywords
Closed

[2.9] Added missing lowercase keywords#1491
michalbundyra wants to merge 1 commit intosquizlabs:2.9from
michalbundyra:hotfix/lowercase-keywords

Conversation

@michalbundyra
Copy link
Copy Markdown
Contributor

  • function (closure)
  • parent
  • self
  • yield

  - function (closure)
  - parent
  - self
  - yield
@jrfnl
Copy link
Copy Markdown
Contributor

jrfnl commented Jun 17, 2017

Unit tests are failing because the yield keyword did not exist yet in PHP < 5.5 and by the looks of it, is not backfilled by PHPCS ?

@gsherwood
Copy link
Copy Markdown
Member

Sorry, missed this 2.9 version.

I don't want to make changes to version 2 unless there are serious bugs, so I'm not going to merge this in as it changes behaviour. To work properly, it also needs the yield backfill, which I wont be doing in version 2 either.

I do have the 3.x PR on the todo though. Thanks.

@gsherwood gsherwood closed this Jun 19, 2017
@michalbundyra michalbundyra deleted the hotfix/lowercase-keywords branch July 5, 2017 23:01
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.

3 participants