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

895 Parameters with default values: allow empty sequences #901

Merged
merged 3 commits into from
Jan 9, 2024

Conversation

ChristianGruen
Copy link
Contributor

Closes #895

@ChristianGruen ChristianGruen added the Tests Needed Tests need to be written or merged label Dec 13, 2023
@ChristianGruen ChristianGruen added Editorial Minor typos, wording clarifications, example fixes, etc. Propose Merge without Discussion Change is editorial or minor labels Dec 16, 2023
@ndw
Copy link
Contributor

ndw commented Jan 9, 2024

The CG agreed to merge this PR without discussion at meeting 060

@ndw ndw merged commit 763c832 into qt4cg:master Jan 9, 2024
2 checks passed
@ChristianGruen ChristianGruen deleted the 895 branch January 10, 2024 11:55
@ChristianGruen ChristianGruen removed the Tests Needed Tests need to be written or merged label Jan 16, 2024
@michaelhkay michaelhkay added Tests Added Tests have been added to the test suites Completed PR has been applied, tests written and tagged, no further action needed and removed Editorial Minor typos, wording clarifications, example fixes, etc. Propose Merge without Discussion Change is editorial or minor labels Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Completed PR has been applied, tests written and tagged, no further action needed Tests Added Tests have been added to the test suites
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Parameters with default values: allow empty sequences
3 participants