Skip to content

Conversation

@sfc-gh-jcieslak
Copy link
Member

@sfc-gh-jcieslak sfc-gh-jcieslak commented Nov 17, 2025

Add listing data source:

  • Add listing data source schema
  • Acceptance tests
  • Documentation (Migration guide entry, examples, data source documentation page template)
  • Preview feature flag + test

@github-actions
Copy link

Integration tests cancelled for 885d9788fc85aecff4ed64f01092273e56289878

@sfc-gh-jcieslak sfc-gh-jcieslak marked this pull request as ready for review November 17, 2025 13:33
@sfc-gh-jcieslak sfc-gh-jcieslak requested a review from a team as a code owner November 17, 2025 13:33
@github-actions
Copy link

Integration tests cancelled for a3f7b122949bd4fbfbbbddb4c7c91c02d58431f6

@github-actions
Copy link

Account-level tests success for 885d9788fc85aecff4ed64f01092273e56289878

@github-actions
Copy link

Account-level tests success for d916e4230f81c4597b70ed618c8527fd006910b2

@github-actions
Copy link

Integration tests failure for d916e4230f81c4597b70ed618c8527fd006910b2

@github-actions
Copy link

Integration tests failure for 7b1fd0bd4578877a82f9dc86320f65e90d83370f

# Conflicts:
#	MIGRATION_GUIDE.md
#	docs/index.md
@github-actions
Copy link

Account-level tests success for 7c633a5730728d88806b1243b9a52211853b109b

@github-actions
Copy link

Integration tests failure for 7c633a5730728d88806b1243b9a52211853b109b

# Conflicts:
#	MIGRATION_GUIDE.md
# Conflicts:
#	MIGRATION_GUIDE.md
sfc-gh-asawicki
sfc-gh-asawicki previously approved these changes Dec 8, 2025
@github-actions
Copy link

github-actions bot commented Dec 8, 2025

Account-level tests failure for a5aa26d9cd5d6c569394e7e25014863eed6f41e9

@github-actions
Copy link

github-actions bot commented Dec 8, 2025

Integration tests success for a5aa26d9cd5d6c569394e7e25014863eed6f41e9

# Conflicts:
#	MIGRATION_GUIDE.md
@github-actions
Copy link

github-actions bot commented Dec 9, 2025

Integration tests cancelled for f4e47735871b0d133583c85caea40d5471ad78df

@github-actions
Copy link

github-actions bot commented Dec 9, 2025

Account-level tests cancelled for f4e47735871b0d133583c85caea40d5471ad78df

Type: schema.TypeBool,
Optional: true,
Default: true,
Description: "Runs DESC LISTING for each listing returned by SHOW LISTINGS. The output of describe is saved to the description field. By default this value is set to true.",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documentation inconsistency that could confuse users: The description states the output is "saved to the description field" but it's actually saved to the describe_output field (as defined on line 38). This should be corrected to:

Description: "Runs DESC LISTING for each listing returned by SHOW LISTINGS. The output of describe is saved to the describe_output field. By default this value is set to true.",
Suggested change
Description: "Runs DESC LISTING for each listing returned by SHOW LISTINGS. The output of describe is saved to the description field. By default this value is set to true.",
Description: "Runs DESC LISTING for each listing returned by SHOW LISTINGS. The output of describe is saved to the describe_output field. By default this value is set to true.",

Spotted by Graphite Agent

Fix in Graphite


Is this helpful? React 👍 or 👎 to let us know.

@sfc-gh-jcieslak sfc-gh-jcieslak merged commit 20b2826 into dev Dec 9, 2025
10 of 12 checks passed
@sfc-gh-jcieslak sfc-gh-jcieslak deleted the listing-data-source branch December 9, 2025 09:09
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.

4 participants