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

Statix: support for traces and term depth configuration #75

Closed
AZWN opened this issue Oct 5, 2021 · 3 comments
Closed

Statix: support for traces and term depth configuration #75

AZWN opened this issue Oct 5, 2021 · 3 comments
Labels
C-Enhancement Category: enhancement. Improvement

Comments

@AZWN
Copy link
Contributor

AZWN commented Oct 5, 2021

In order to debug Statix specifications, it would be useful to be able to control the stack trace length and the term depth of the messages that are generated.

@AZWN AZWN added S-Needs-Triage Status: needs labels C-Enhancement Category: enhancement. Improvement labels Oct 5, 2021
@Gohla Gohla removed the S-Needs-Triage Status: needs labels label Oct 5, 2021
@Gohla
Copy link
Member

Gohla commented Oct 5, 2021

@AZWN In Spoofax 2, this is configurable per-project right? We do not yet have project-specific settings in Spoofax 3, and this is something that I want to spend some design time on to get it right in Spoofax 3. Would it be fine for now to have this configurable per-language, and overridable per-project in the future? And what do you think the defaults should be?

@AZWN
Copy link
Contributor Author

AZWN commented Oct 5, 2021

Yes, a per-language solution would be fine for now. The Spoofax 2 defaults are: 0 for stack trace length, and 3 for term depth.

@Gohla
Copy link
Member

Gohla commented Oct 5, 2021

Implemented in 21fc3f3

@Gohla Gohla closed this as completed Oct 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Enhancement Category: enhancement. Improvement
Projects
None yet
Development

No branches or pull requests

2 participants