Skip to content

Conversation

@Lotes
Copy link
Contributor

@Lotes Lotes commented Nov 4, 2025

Closes #283

I added two sub pages:

  • the classical solution
  • the infix syntax solution.

Just for the case, when someone needs more control over his operator rules.

Anything to add?

@Lotes Lotes requested a review from msujew November 4, 2025 11:17
@Lotes Lotes temporarily deployed to pull-request-preview November 4, 2025 11:18 — with GitHub Actions Inactive
@github-actions
Copy link

github-actions bot commented Nov 4, 2025

PR Preview Action v1.6.1
Preview removed because the pull request was closed.
2025-11-18 16:04 UTC

@Lotes Lotes temporarily deployed to pull-request-preview November 4, 2025 12:45 — with GitHub Actions Inactive
@spoenemann
Copy link
Contributor

Shouldn't the documentation of this grammar feature primarily be done in the Grammar Language reference?

@Lotes Lotes temporarily deployed to pull-request-preview November 5, 2025 09:18 — with GitHub Actions Inactive
@Lotes Lotes temporarily deployed to pull-request-preview November 5, 2025 09:38 — with GitHub Actions Inactive
Copy link
Contributor

@spoenemann spoenemann left a comment

Choose a reason for hiding this comment

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

Looks good, thanks. Could you also have a look at the "Tree-Rewriting Actions" section of the Grammar Reference and add hints to the more detailed documentation where appropriate?

@Lotes Lotes temporarily deployed to pull-request-preview November 18, 2025 10:49 — with GitHub Actions Inactive
Copy link
Contributor

@spoenemann spoenemann left a comment

Choose a reason for hiding this comment

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

👍

@Lotes Lotes temporarily deployed to pull-request-preview November 18, 2025 15:46 — with GitHub Actions Inactive
@Lotes Lotes merged commit 7b3505e into main Nov 18, 2025
8 checks passed
@Lotes Lotes deleted the lotes/operators branch November 18, 2025 16:04
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.

Add documentation: Infix Operator Rules

3 participants