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

Sysl parser doesn't populate source_context for stmt #990

Open
orlade-anz opened this issue Aug 3, 2020 · 2 comments
Open

Sysl parser doesn't populate source_context for stmt #990

orlade-anz opened this issue Aug 3, 2020 · 2 comments
Labels
bug P0 Highest priority: important or urgent

Comments

@orlade-anz
Copy link
Collaborator

No description provided.

@orlade-anz orlade-anz added bug P0 Highest priority: important or urgent labels Aug 3, 2020
@springwiz
Copy link
Contributor

springwiz commented Oct 1, 2020

@orlade-anz Can you shed some light on this issue? What exactly is the need here? I see that there is already a source context associated with stmt in the sysl protobuf. for eg.

     "stmt": [......],
     "sourceContext": {
      "file": "args.sysl",
      "start": {
       "line": 12,
       "col": 4
      },
      "end": {
       "line": 17
      }
     }
    }
   },

@orlade-anz
Copy link
Collaborator Author

Each element of "stmt": [......] should have a source_context of its own.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug P0 Highest priority: important or urgent
Projects
None yet
Development

No branches or pull requests

2 participants