Skip to content

fix: Bump sjson library to fix accessing root arrays and complex path#21022

Merged
kodiakhq[bot] merged 1 commit intomainfrom
feature/eng-1733-enable-delete-with-nested-arrays-in-sjson-library-and-basic
Jul 15, 2025
Merged

fix: Bump sjson library to fix accessing root arrays and complex path#21022
kodiakhq[bot] merged 1 commit intomainfrom
feature/eng-1733-enable-delete-with-nested-arrays-in-sjson-library-and-basic

Conversation

@przste-go
Copy link
Copy Markdown

@przste-go przste-go commented Jul 15, 2025

This PR fixes remove_column to support removing json arrays with 'foo.#.bar' syntax. It also fixes accessing complex paths starting from '#' in example '#.env.#.value'

Summary

⚠️ If you're contributing to a plugin please read this section of the contribution guidelines 🧑‍🎓 before submitting this PR ⚠️

@przste-go przste-go requested review from a team and blesniewski July 15, 2025 10:30
@przste-go przste-go changed the title fix: Bump sjson library to fix accessing root arrays and complex path… fix: Bump sjson library to fix accessing root arrays and complex path Jul 15, 2025
@przste-go przste-go added the automerge Automatically merge once required checks pass label Jul 15, 2025
@kodiakhq kodiakhq bot merged commit 4f16d35 into main Jul 15, 2025
29 checks passed
@kodiakhq kodiakhq bot deleted the feature/eng-1733-enable-delete-with-nested-arrays-in-sjson-library-and-basic branch July 15, 2025 10:48
kodiakhq bot pushed a commit that referenced this pull request Jul 15, 2025
🤖 I have created a release *beep* *boop*
---


## [2.6.3](plugins-transformer-basic-v2.6.2...plugins-transformer-basic-v2.6.3) (2025-07-15)


### Bug Fixes

* Bump sjson library to fix accessing root arrays and complex path ([#21022](#21022)) ([4f16d35](4f16d35))
* **deps:** Update github.com/cloudquery/sjson digest to 737185f ([#21023](#21023)) ([60f00a1](60f00a1))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants