docs: correct stale doc comments in pq context and emacs README#6069
Open
prql-bot wants to merge 1 commit into
Open
docs: correct stale doc comments in pq context and emacs README#6069prql-bot wants to merge 1 commit into
prql-bot wants to merge 1 commit into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two stale/incorrect documentation fixes found in the nightly code-quality sweep. Comment/markdown only — no code behavior changes.
prqlc/prqlc/src/sql/pq/context.rs: the module-level docstring described semantic resolution ("Transform the parsed AST into a materialized AST, by executing functions and replacing variables"), which is the behavior of the semantic layer, not this file. This module definesAnchorContext, the PQ anchoring context that tracks table/column/relation-instance declarations and generates fresh IDs and names during RQ→SQL lowering. Rewrote the docstring to match, consistent with the struct's own doc comment just below it.grammars/emacs/README.md: the install-snippet code fence used```emacs, which is not a recognized highlight language, so renderers fall back to plain text. Changed to```elisp.No tests apply — these are documentation-only corrections.