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

feat: Option to disable LSPP defaults #649

Merged
merged 4 commits into from
Jan 3, 2025
Merged

feat: Option to disable LSPP defaults #649

merged 4 commits into from
Jan 3, 2025

Conversation

koubaa
Copy link
Collaborator

@koubaa koubaa commented Jan 3, 2025

Fixes #594

Add an option to disable all defaults, forcing the user to have to set all fields themselves.

This changed all auto-generated keyword files. To review the codegen changes, review these two commits:
8945ae9
6856fba

Mohamed Koubaa added 3 commits January 3, 2025 12:55
use the option in one keyword
add contextmanager to scope disabling
@koubaa koubaa requested a review from a team as a code owner January 3, 2025 19:01
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Jan 3, 2025
@koubaa koubaa merged commit 183e787 into main Jan 3, 2025
28 of 29 checks passed
@koubaa koubaa deleted the no-defaults branch January 3, 2025 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Consider a global flag not to include LSPP defaults
2 participants