Skip to content

chore: bump keboola-json-to-csv to 0.0.13#207

Merged
ZdenekSrotyr merged 1 commit into
masterfrom
devin/1773085916-bump-json-to-csv-0.0.13
Mar 11, 2026
Merged

chore: bump keboola-json-to-csv to 0.0.13#207
ZdenekSrotyr merged 1 commit into
masterfrom
devin/1773085916-bump-json-to-csv-0.0.13

Conversation

@Jakuboola

@Jakuboola Jakuboola commented Mar 9, 2026

Copy link
Copy Markdown
Collaborator

chore: bump keboola-json-to-csv to 0.0.13

Summary

Bumps keboola-json-to-csv from 0.0.12 to 0.0.13 in python-sync-actions. Version 0.0.13 contains the fix for SUPPORT-15504 — a TypeError: argument of type 'NodeType' is not iterable crash in the "Infer Mapping" sync action caused by using in instead of == for an enum comparison in the JSON analyzer.

Upstream fix PR: keboola/python-json-to-csv#14

Review & Testing Checklist for Human

  • Test "Infer Mapping" with polymorphic JSON: The fix enables a previously-unreachable code path (DICT → LIST type upgrade). Test with an API response where the same field is a dict in one row and a list in another to verify the crash no longer occurs and the inferred mapping is correct.
  • Verify 0.0.13 is available on PyPI

Notes


Open with Devin

Co-Authored-By: Jakub Sochan <jakub.sochan@keboola.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@ZdenekSrotyr ZdenekSrotyr self-requested a review March 11, 2026 09:52

@ZdenekSrotyr ZdenekSrotyr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

OK

@soustruh soustruh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM ദ്ദി(•ᴗ•)

@ZdenekSrotyr ZdenekSrotyr merged commit ca7ca98 into master Mar 11, 2026
7 checks passed
@ZdenekSrotyr ZdenekSrotyr deleted the devin/1773085916-bump-json-to-csv-0.0.13 branch March 11, 2026 10:15
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