You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The parser doesn't currently allow for that, because it treats NamespaceScope as an Identifier, which only allows for alpha numeric characters, dots, and underlines: [a-zA-Z0-9\._].
The solution would be to create a NamespaceScope type, which allows for Identifier | *. This would be a breaking change, though. Because it would changes the AST output.
I'm happy to work on a PR for that, if you are ok with the breaking change, and you don't have any other alternative solution to fix this issue.
The text was updated successfully, but these errors were encountered:
The Thrift spec allows you to use
namespace *
, which means that the IDL applies to all target languages: https://thrift.apache.org/docs/idl#namespaceThe parser doesn't currently allow for that, because it treats
NamespaceScope
as anIdentifier
, which only allows for alpha numeric characters, dots, and underlines:[a-zA-Z0-9\._]
.The solution would be to create a
NamespaceScope
type, which allows forIdentifier | *
. This would be a breaking change, though. Because it would changes the AST output.I'm happy to work on a PR for that, if you are ok with the breaking change, and you don't have any other alternative solution to fix this issue.
The text was updated successfully, but these errors were encountered: