Skip to content

Fix yaml parse issue#59

Merged
gowthamrajj merged 3 commits into
Workday:v1.8.0from
gowthamrajj:fix/yaml-issue
Jun 10, 2026
Merged

Fix yaml parse issue#59
gowthamrajj merged 3 commits into
Workday:v1.8.0from
gowthamrajj:fix/yaml-issue

Conversation

@gowthamrajj

Copy link
Copy Markdown
Contributor

Changes

  • YAML reserved tokens round-trip safely. Values like OFF, ON, YES, NO (and lowercase variants) are now quoted on emit and tolerated on load, so time_intervals: OFF no longer turns into false in the manifest and crashes sync. Per-column meta failures also name the offending column.
  • Sync errors surface the real cause. SQL/YML generation failures now show the underlying message instead of always pointing at expr syntax.

@arunxdev

arunxdev commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@gowthamrajj gowthamrajj changed the base branch from main to v1.8.0 June 9, 2026 16:01
@gowthamrajj gowthamrajj merged commit e4d43fd into Workday:v1.8.0 Jun 10, 2026
3 of 5 checks passed
@gowthamrajj gowthamrajj deleted the fix/yaml-issue branch June 10, 2026 07:10
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.

3 participants