Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: Improve readability (for Alternative operator '//') #2898

Merged
merged 1 commit into from
Dec 11, 2023

Conversation

jesperronn
Copy link
Contributor

The description of the Alternative operator // was hard for me to grasp in its wording.

I suggest dividing the looong sentence into two parts.

Reviewers: Please let me know if I understand the funcitonality correctly

@jesperronn jesperronn changed the title chore: Improve readability (for Alternative operator '//' chore: Improve readability (for Alternative operator '//') Sep 15, 2023
@itchyny itchyny added the docs label Sep 17, 2023
@wader
Copy link
Member

wader commented Sep 18, 2023

Hi, i think you might need to run make jq.1.prebuilt to fix the CI error.

@emanuele6
Copy link
Member

Thank you! I will regenerate the manual for you

@emanuele6
Copy link
Member

Hmm, I need to fix conflicts now

The description of the Alternative operator `//` was hard for me to grasp in its wording.

I suggest dividing the looong sentence into two parts. Since it is actually an alternative formulation, the sentence can be divided into the first formulation, and the alternative formulation.
@emanuele6 emanuele6 enabled auto-merge (squash) December 11, 2023 11:34
@emanuele6 emanuele6 merged commit 2ccab04 into jqlang:master Dec 11, 2023
@jesperronn jesperronn deleted the patch-1 branch December 13, 2023 15:44
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants