Skip to content

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

Merged
ZdenekSrotyr merged 1 commit intomasterfrom
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 intomasterfrom
devin/1773085916-bump-json-to-csv-0.0.13

Conversation

@Jakuboola
Copy link
Copy Markdown
Contributor

@Jakuboola Jakuboola commented Mar 9, 2026

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

Copy link
Copy Markdown
Contributor

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

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
Copy link
Copy Markdown
Contributor

@ZdenekSrotyr ZdenekSrotyr left a comment

Choose a reason for hiding this comment

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

OK

Copy link
Copy Markdown
Contributor

@soustruh soustruh left a comment

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