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

Improve stylesheet for generating keyword tests #842

Merged
merged 1 commit into from Nov 21, 2023

Conversation

michaelhkay
Copy link
Contributor

Improves the stylesheet that generates the test set misc/BuiltInKeywords.xml; specifically, it's smarter about generating acceptable callback functions that won't trigger an unwanted error.

(The generated test calls each function twice, once with positional arguments and once with keyword arguments, and checks that the two results are deep-equal).

@michaelhkay michaelhkay added XQFO An issue related to Functions and Operators Editorial Minor typos, wording clarifications, example fixes, etc. labels Nov 17, 2023
@ChristianGruen ChristianGruen added the Propose Merge without Discussion Change is editorial or minor label Nov 17, 2023
@ndw
Copy link
Contributor

ndw commented Nov 21, 2023

The group agreed to merge this PR without discussion at meeting 055.

@ndw ndw merged commit ab946b1 into qt4cg:master Nov 21, 2023
2 checks passed
@ChristianGruen ChristianGruen removed the Propose Merge without Discussion Change is editorial or minor label Jan 2, 2024
@michaelhkay michaelhkay added Completed PR has been applied, tests written and tagged, no further action needed Build Affects the build infrastructure only and removed XQFO An issue related to Functions and Operators Editorial Minor typos, wording clarifications, example fixes, etc. Completed PR has been applied, tests written and tagged, no further action needed labels Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Affects the build infrastructure only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants