Skip to content

WIP: Port to Antlr4#4

Open
kasbah wants to merge 64 commits into
devfrom
antlr
Open

WIP: Port to Antlr4#4
kasbah wants to merge 64 commits into
devfrom
antlr

Conversation

@kasbah
Copy link
Copy Markdown
Member

@kasbah kasbah commented Jan 21, 2018

Things required to merge this:

  • Full feature parity with previous electro-grammar
  • Port the demo site
  • Update documentation

@kasbah kasbah force-pushed the antlr branch 8 times, most recently from f95d073 to 38d8465 Compare January 21, 2018 22:08
dvc94ch and others added 6 commits January 23, 2018 19:59
Renames characteristic -> dielectric.
Renames size -> package_size.
Only uses reasonable units (1F -> 1uF).
Uncomments tests that fail because of tolerance position.
@kasbah kasbah changed the base branch from master to dev January 25, 2018 21:00
@dvc94ch
Copy link
Copy Markdown
Collaborator

dvc94ch commented Feb 3, 2018

@kasbah Mmh not sure why it's failing the style checker. When I run them on my computer the git diff is empty. I'm also not sure why it passed with that semicolon before, since it's been there for a while...

@kasbah
Copy link
Copy Markdown
Member Author

kasbah commented Feb 4, 2018

There was a diff in the package-lock.json. I fixed this by commiting the yarn.lock so travis will use yarn. I use yarn myself anyway. Do you mind using it?

@dvc94ch
Copy link
Copy Markdown
Collaborator

dvc94ch commented Feb 4, 2018 via email

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.

2 participants