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.

Sign up for GitHub

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

Jump to bottom

Add mechanism for easily adding tests for command-line options #4907

Open
kfcripps opened this issue Sep 11, 2024 · 2 comments · May be fixed by #5175
Open

Add mechanism for easily adding tests for command-line options #4907

kfcripps opened this issue Sep 11, 2024 · 2 comments · May be fixed by #5175
Assignees
@kfcripps
Labels
infrastructure Topics related to code style and build and test infrastructure.

Comments

@kfcripps
Copy link
Contributor

e.g. --Werror, --Wdisable, etc.

See #4894 and #4366 for initial test ideas.
@kfcripps kfcripps added the infrastructure Topics related to code style and build and test infrastructure. label Sep 11, 2024
@kfcripps kfcripps mentioned this issue Sep 11, 2024
Merged
@kfcripps
Copy link
Contributor Author

@sepanchi JFYI - This may be a good one for you to work on if you have time :)

@kfcripps kfcripps mentioned this issue Feb 19, 2025
Merged
@kfcripps kfcripps linked a pull request Mar 12, 2025 that will close this issue
Open
@kfcripps kfcripps assigned kfcripps and unassigned sepanchi Mar 12, 2025
@kfcripps
Copy link
Contributor Author

#5123 added such a mechanism and #5175 adds tests for functionality that was added by PRs #4366 and #4894.

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

@kfcripps kfcripps

Labels
infrastructure Topics related to code style and build and test infrastructure.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#4365] [#4907] Add test cases related to issue #4365 kfcripps/p4c
2 participants
@kfcripps @sepanchi