Skip to content

Allow slimit to work with ply 3.6#79

Open
moses-palmer wants to merge 1 commit intorspivak:masterfrom
moses-palmer:fixup-ply-lextab
Open

Allow slimit to work with ply 3.6#79
moses-palmer wants to merge 1 commit intorspivak:masterfrom
moses-palmer:fixup-ply-lextab

Conversation

@moses-palmer
Copy link

Always set slimit.parser.*tab to strings

These values are expected by ply to be module names, not module
instances. ply 3.6 will crash if actual modules are passed.

Please note that the actual tab modules depend on the version of ply first used. Those modules are checked in, and not modified by this commit.

These values are expected by ply to be module names, not module
instances. ply 3.6 will crash if actual modules are passed.
@delijati
Copy link

Could this be please merged

Jormangeud added a commit to Jormangeud/jrg-overlay that referenced this pull request Jul 29, 2016
metatoaster added a commit to calmjs/calmjs.parse that referenced this pull request Jun 8, 2017
- Also note that the following issues were addressed, where applicable
  to the lexer or parser.

  - rspivak/slimit#52
  - rspivak/slimit#54
  - rspivak/slimit#57
  - rspivak/slimit#59
  - rspivak/slimit#62
  - rspivak/slimit#65
  - rspivak/slimit#70
  - rspivak/slimit#73
  - rspivak/slimit#79
  - rspivak/slimit#81
  - rspivak/slimit#82
  - rspivak/slimit#90

- Will get the release out when I get some sleep.
@ale-rt
Copy link
Contributor

ale-rt commented Aug 8, 2018

I think this one has been superseded by #102

Jormangeud added a commit to Jormangeud/jrg-overlay that referenced this pull request May 8, 2023
Jormangeud added a commit to Jormangeud/jrg-overlay that referenced this pull request May 8, 2023
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