Skip to content

Update function explanations to new syntax#3811

Merged
Drodt merged 1 commit intomainfrom
function-explanation
Apr 22, 2026
Merged

Update function explanations to new syntax#3811
Drodt merged 1 commit intomainfrom
function-explanation

Conversation

@Drodt
Copy link
Copy Markdown
Member

@Drodt Drodt commented Apr 21, 2026

Intended Change

The syntax in functionsExplanations.xml was out of date and had a minor syntax error. This PR brings it up do date.

Type of pull request

  • Bug fix (non-breaking change which fixes an issue)
  • Other: Changes to XML which is read by GUI

Ensuring quality

  • I have tested the feature as follows: I started KeY

Additional information and contact(s)

The contributions within this pull request are licensed under GPLv2 (only) for inclusion in KeY.

@Drodt Drodt self-assigned this Apr 21, 2026
@Drodt Drodt added Review Request Waiting for review 🛠 Maintenance Code quality and related things w/o functional changes labels Apr 21, 2026
@unp1 unp1 self-requested a review April 22, 2026 03:57
Copy link
Copy Markdown
Member

@unp1 unp1 left a comment

Choose a reason for hiding this comment

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

thanks

@Drodt Drodt added this pull request to the merge queue Apr 22, 2026
Merged via the queue into main with commit de3eae7 Apr 22, 2026
36 checks passed
@Drodt Drodt deleted the function-explanation branch April 22, 2026 06:42
@wadoon
Copy link
Copy Markdown
Member

wadoon commented Apr 22, 2026

With #3787, the functionsExplanations.xml will die in favor of inline comments in KeY files.

With #3786, the documentation will be presented in the UI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🛠 Maintenance Code quality and related things w/o functional changes Review Request Waiting for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants