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

docs: Fix operators table #4657

Closed
wants to merge 5 commits into from

Conversation

richb-hanover
Copy link
Contributor

Fixed table for the operator precedence in Operators.md

@max-sixty
Copy link
Member

pow is part of #4125 !

Will merge tomorrow unless I get feedback.

Thank you for spotting though!

@max-sixty
Copy link
Member

Will close since #4125 is merged. Thank you regardless!

@max-sixty max-sixty closed this Jun 24, 2024
@richb-hanover
Copy link
Contributor Author

@max-sixty I'm not sure of the state of this, but the table of Operator Precedence (https://prql-lang.org/book/reference/syntax/operators.html#operator-precedence) in the public Book is still messed up. Would you check? Thanks

@max-sixty
Copy link
Member

Yes, we should do a release. (Generally we push docs-only changes to web, but here it was both docs and code...)

In retrospect I should have merged yours, backported to web, and then resolved the conflicts with mine. Sorry!

@richb-hanover
Copy link
Contributor Author

Ahah! I see now... Both code and docs. I shoulda made separate PRs. I'll do that next time... Thanks

@max-sixty
Copy link
Member

Ahah! I see now... Both code and docs. I shoulda made separate PRs. I'll do that next time... Thanks

Yours was perfect! Because this was docs only. So mea culpa...

@richb-hanover
Copy link
Contributor Author

I don't do "blame". It's enough if it gets fixed when it's pointed out. See WPNTFIIWDKIB

@richb-hanover richb-hanover deleted the fix-operators-table branch August 16, 2024 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants