Skip to content

Create a script to simplify the language specification#4340

Merged
eernstg merged 64 commits into
mainfrom
simplify_spec_apr25
May 9, 2025
Merged

Create a script to simplify the language specification#4340
eernstg merged 64 commits into
mainfrom
simplify_spec_apr25

Conversation

@eernstg
Copy link
Copy Markdown
Member

@eernstg eernstg commented Apr 27, 2025

The language specification uses semantic line breaks and rather short lines. This means that it gets somewhat inconvenient to search in a number of situations. For example, we may find the command \Error{compile-time error} on one line and the text that describes which error it is on the next line. Also, when searching the language specification for hard facts it may create some noise that a rather large portion of the language specification is commentary, rationale, and comments.

This PR adds a small script that processes the language specification such that it is more convenient to search for hard facts: It removes comments, commentary, and rationale, and it joins paragraphs written as several lines into a single line. This terse version of the language specification can be created by invoking make terse.

Copy link
Copy Markdown
Contributor

@chloestefantsova chloestefantsova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with a few nits.

Comment thread specification/scripts/simplify_specification.dart Outdated
Comment thread specification/scripts/simplify_specification.dart Outdated
Comment thread specification/scripts/simplify_specification.dart Outdated
Comment thread specification/scripts/simplify_specification.dart Outdated
@eernstg
Copy link
Copy Markdown
Member Author

eernstg commented Apr 28, 2025

Thanks!

@eernstg eernstg merged commit d8e4240 into main May 9, 2025
4 checks passed
@eernstg eernstg deleted the simplify_spec_apr25 branch May 9, 2025 06:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants