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

Move an example to the manual and regenerate man.test #2639

Merged
merged 1 commit into from
Jun 29, 2023

Conversation

itchyny
Copy link
Contributor

@itchyny itchyny commented Jun 28, 2023

The PR #2400 added a test to man.test, but this file is generated from the manual. There're two options; move this test to jq.test or to the manual. I think this is worth adding to the ? operator section, so added there and regenerated the test file.

@itchyny itchyny added the test label Jun 28, 2023
@itchyny itchyny added this to the 1.7 release milestone Jun 28, 2023
@wader
Copy link
Member

wader commented Jun 29, 2023

Makes sene to me. The other changes to man.test are because we haven't regenerated it a while?

@itchyny
Copy link
Contributor Author

itchyny commented Jun 29, 2023

Yeah, we could generate it and make CI fail if there is a diff.

@itchyny itchyny merged commit f88c4e5 into jqlang:master Jun 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants