Skip to content

Fix command add_args#354

Open
PearsonWhite wants to merge 1 commit into
masterfrom
pwhite/command
Open

Fix command add_args#354
PearsonWhite wants to merge 1 commit into
masterfrom
pwhite/command

Conversation

@PearsonWhite

Copy link
Copy Markdown
Contributor

Add_args was comparing the flag to tuples when searching for existing args, thus failing to properly replace when on_duplicate="replace".

Add_args was comparing the flag to tuples when searching for existing
args, thus failing to properly replace when on_duplicate="replace".
@PearsonWhite PearsonWhite self-assigned this Jul 16, 2026
@PearsonWhite PearsonWhite added this to DST Jul 16, 2026
@PearsonWhite PearsonWhite added the ift IFT commitments label Jul 16, 2026
@PearsonWhite
PearsonWhite marked this pull request as ready for review July 16, 2026 13:14

@radiken radiken 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!

@AlbertoSoutullo AlbertoSoutullo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ift IFT commitments

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants