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

feat: make api-linter more debuggable #1086

Closed
shwoodard opened this issue Feb 9, 2023 · 1 comment · Fixed by #1087 or #1088
Closed

feat: make api-linter more debuggable #1086

shwoodard opened this issue Feb 9, 2023 · 1 comment · Fixed by #1087 or #1088
Labels
priority: p3 type: feature request

Comments

@shwoodard
Copy link
Collaborator 

$ api-linter service.proto
2023/02/08 19:10:09 runtime error: index out of range [1] with length 1

Very hard to debug!

Make it easier.
shwoodard added a commit to shwoodard/api-linter that referenced this issue Feb 9, 2023
@shwoodard
feat: enable better debugability for linting
8dabb1e 
Adds a debug flag to the cli and passes it to lint.Linter. If enabled,
when a panic is recoverd, the stack will be printed by the runtime/debug
package.

Closes googleapis#1086
@shwoodard shwoodard mentioned this issue Feb 9, 2023
Merged
toumorokoshi pushed a commit that referenced this issue Feb 9, 2023
@shwoodard
feat: enable better debugability for linting (#1087)
16fc88e 
Adds a debug flag to the cli and passes it to lint.Linter. If enabled,
when a panic is recoverd, the stack will be printed by the runtime/debug
package.

Closes #1086
@shwoodard
Copy link
Collaborator Author

Needs tests.

@shwoodard shwoodard reopened this Feb 9, 2023
@shwoodard shwoodard mentioned this issue Feb 9, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
priority: p3 type: feature request
Projects
None yet
Milestone
No milestone
1 participant
@shwoodard