Skip to content

Conversation

@shangyian
Copy link
Collaborator

@shangyian shangyian commented Jan 17, 2026

Summary

This fixes the YAML export of nodes to have queries that are multiline rather than a single string separated by newlines.

The original code was modifying the yaml.SafeDumper class directly, which didn't reliably apply the multiline representer. The fix creates a new subclass each time. Now queries with newlines will be exported as:

  query: |
    select
    *
    FROM ...

Instead of:

  query: "select\n*\nFROM ..."

Test Plan

Deployment Plan

N/A

@netlify
Copy link

netlify bot commented Jan 17, 2026

Deploy Preview for thriving-cassata-78ae72 canceled.

Name Link
🔨 Latest commit e99a0dc
🔍 Latest deploy log https://app.netlify.com/projects/thriving-cassata-78ae72/deploys/696bb4b0072bda0008979e9c

@shangyian shangyian marked this pull request as ready for review January 17, 2026 17:21
@shangyian shangyian requested a review from anhqle January 17, 2026 17:21
@shangyian shangyian merged commit 81d107b into DataJunction:main Jan 17, 2026
17 checks passed
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.

1 participant