Skip to content

Improve error message for top-level arrays#2138

Merged
felixfontein merged 3 commits intogetsops:mainfrom
kushal9897:fix/json-array-error-message
Apr 7, 2026
Merged

Improve error message for top-level arrays#2138
felixfontein merged 3 commits intogetsops:mainfrom
kushal9897:fix/json-array-error-message

Conversation

@kushal9897
Copy link
Copy Markdown
Contributor

…#1974)

Signed-off-by: Kushal Agrawal <153362678+kushal9897@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution!

Comment thread README.rst Outdated
@felixfontein felixfontein changed the title fix(json): improve error message for top-level arrays (fixes #1974) Improve error message for top-level arrays (fixes #1974) Apr 6, 2026
@felixfontein felixfontein changed the title Improve error message for top-level arrays (fixes #1974) Improve error message for top-level arrays Apr 6, 2026
@kushal9897
Copy link
Copy Markdown
Contributor Author

Thanks for the feedback!

That makes sense I’ll remove this from the indentation section.

Signed-off-by: Kushal Agrawal <153362678+kushal9897@users.noreply.github.com>
@kushal9897 kushal9897 force-pushed the fix/json-array-error-message branch from 50d9f1d to 7ec9470 Compare April 6, 2026 21:37
@felixfontein felixfontein merged commit f082bec into getsops:main Apr 7, 2026
20 checks passed
@felixfontein
Copy link
Copy Markdown
Contributor

@kushal9897 thanks for your contribution!

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