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

Drop browser tests #2755

Merged
merged 5 commits into from
Feb 22, 2020
Merged

Drop browser tests #2755

merged 5 commits into from
Feb 22, 2020

Conversation

ericvergnaud
Copy link
Contributor

No description provided.

…n in our CI, everybody uses some packaging, next official js runtime will be pre-packaged

overall not worth the maintenance effort anymore
@ericvergnaud
Copy link
Contributor Author

@parrt I might need your help with this one...
this is only about javascript testing
mvn test script works fine locally, but fails on travis, complaining it's not finding any tests
not sure it that has to do with some test generation config?

@ericvergnaud
Copy link
Contributor Author

@parrt forget about the help, pb fixed
Not sure why git decided to ignore those test files when I renamed the package...
Good to merge

@parrt parrt merged commit 2d9f45d into antlr:master Feb 22, 2020
@ericvergnaud ericvergnaud deleted the drop-browser-tests branch February 23, 2020 02:30
@carocad carocad mentioned this pull request Oct 4, 2020
@parrt parrt added this to the 4.9 milestone Nov 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants