Skip to content

Conversation

julienrbrt
Copy link
Member

Add xast.ModifyCaller to swap arguments from a function.
This is be required for improving the rollkit app (ignite/apps#192) instead of doing search and replaces.

@julienrbrt julienrbrt added the skip-changelog Don't check changelog for new entries label Apr 18, 2025
@julienrbrt julienrbrt added the backport/v28.x.y Backport to v28.x.y label Apr 18, 2025
@julienrbrt julienrbrt marked this pull request as ready for review April 18, 2025 15:36
@julienrbrt julienrbrt removed the skip-changelog Don't check changelog for new entries label Apr 18, 2025
Co-authored-by: Danilo Pantani <danpantani@gmail.com>
@julienrbrt julienrbrt requested a review from Pantani April 22, 2025 14:22
@julienrbrt julienrbrt merged commit 121c3a8 into main Apr 22, 2025
3 checks passed
@julienrbrt julienrbrt deleted the julien/xast-caller branch April 22, 2025 15:04
mergify bot pushed a commit that referenced this pull request Apr 22, 2025
* feat(pkg/xast): add `ModifyCaller` function

* update modify caller

* lint

* feedback

Co-authored-by: Danilo Pantani <danpantani@gmail.com>

---------

Co-authored-by: Danilo Pantani <danpantani@gmail.com>
(cherry picked from commit 121c3a8)

# Conflicts:
#	ignite/internal/plugin/testdata/execute_fail/go.mod
#	ignite/internal/plugin/testdata/execute_ok/go.mod
#	ignite/pkg/xast/function.go
julienrbrt added a commit that referenced this pull request Apr 22, 2025
* feat(pkg/xast): add `ModifyCaller` function (#4639)

* feat(pkg/xast): add `ModifyCaller` function

* update modify caller

* lint

* feedback

Co-authored-by: Danilo Pantani <danpantani@gmail.com>

---------

Co-authored-by: Danilo Pantani <danpantani@gmail.com>
(cherry picked from commit 121c3a8)

# Conflicts:
#	ignite/internal/plugin/testdata/execute_fail/go.mod
#	ignite/internal/plugin/testdata/execute_ok/go.mod
#	ignite/pkg/xast/function.go

* conflicts

* cl

* up

---------

Co-authored-by: julienrbrt <julien@rbrt.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants