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.

Sign up for GitHub

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

Jump to bottom

[bf-lu] antlr4 produces "INVALID_ALT_NUMBER" warning on node >= 14.2.X #1216

Closed
a-b-r-o-w-n opened this issue Apr 23, 2021 · 0 comments · Fixed by #1254
Closed

[bf-lu] antlr4 produces "INVALID_ALT_NUMBER" warning on node >= 14.2.X #1216

a-b-r-o-w-n opened this issue Apr 23, 2021 · 0 comments · Fixed by #1254
Assignees
@munozemilio
Labels
lu non luis api commands P1 Painful if we don't fix, won't block releasing
Milestone
R14

Comments

@a-b-r-o-w-n
Copy link
Contributor

a-b-r-o-w-n commented Apr 23, 2021 •
edited
Loading

Describe the bug

I'm not sure where this originating, but I think that antlr/antlr4#2834 is the root cause.

I see this when running Composer with node 14.15.x:

(node:94216) Warning: Accessing non-existent property 'INVALID_ALT_NUMBER' of module exports inside circular dependency

Related to #1061

Expected behavior

The issue linked above suggests that this has been fixed in a newer version of antlr4.

Additional Information

This does not seem to break anything, but I'm trying to clean up the node warnings in Composer and came across this, so maybe something to look into during a quality milestone post //build.

[bug]
@a-b-r-o-w-n a-b-r-o-w-n changed the title [bf-lu] antlr4 produces "INVALID_ALT_NUMBER" error on node >= 14.2.X [bf-lu] antlr4 produces "INVALID_ALT_NUMBER" warning on node >= 14.2.X Apr 23, 2021
@munozemilio munozemilio added the lu non luis api commands label Jun 5, 2021
@munozemilio munozemilio added this to the R14 milestone Jun 5, 2021
@munozemilio munozemilio added the P1 Painful if we don't fix, won't block releasing label Jun 5, 2021
@munozemilio munozemilio self-assigned this Jun 7, 2021
@munozemilio munozemilio mentioned this issue Jun 28, 2021
Merged
This was referenced Dec 31, 2021
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
@snyk-bot snyk-bot mentioned this issue Aug 18, 2022
Open
This was referenced Aug 19, 2022
Open
Open
Open
This was referenced Aug 19, 2022
Open
Open
Open
Open
This was referenced Aug 19, 2022
Open
Open
This was referenced Aug 19, 2022
Open
Open
Open
Open
Open
Open
@snyk-bot snyk-bot mentioned this issue Aug 20, 2022
Open
This was referenced Jun 21, 2023
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
This was referenced Nov 30, 2023
Open
Open
Open
Open
Open
Open
Open
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@munozemilio munozemilio

Labels
lu non luis api commands P1 Painful if we don't fix, won't block releasing
Projects
None yet
Milestone
R14
Development

Successfully merging a pull request may close this issue.

Upgrade Antlr to 4.9.2 microsoft/botframework-cli
2 participants
@a-b-r-o-w-n @munozemilio