Skip to content

Comments

fix: raise error when accessing an empty parameter#261

Merged
yrh012 merged 7 commits intomainfrom
fix/empty-param
Feb 5, 2026
Merged

fix: raise error when accessing an empty parameter#261
yrh012 merged 7 commits intomainfrom
fix/empty-param

Conversation

@yrh012
Copy link

@yrh012 yrh012 commented Feb 4, 2026

Description

This PR solves the issue by ...

Review guidelines

Estimated Time of Review: 2 minutes

Checklist before merging:

  • Confirm that the relevant changelog(s) are up-to-date in case of any user-facing changes

@yrh012 yrh012 self-assigned this Feb 4, 2026
@domire8 domire8 changed the title fix: raise exception error when accessing an empty parameter fix: raise error when accessing an empty parameter Feb 4, 2026
Copy link
Member

@domire8 domire8 left a comment

Choose a reason for hiding this comment

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

Looks good

Copy link
Member

@eeberhard eeberhard left a comment

Choose a reason for hiding this comment

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

Seems sensible!

yrh012 and others added 2 commits February 5, 2026 10:00
Co-authored-by: Enrico Eberhard <32450951+eeberhard@users.noreply.github.com>
Copy link
Member

@domire8 domire8 left a comment

Choose a reason for hiding this comment

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

Good to go when the tests are green thanks!

@yrh012 yrh012 merged commit 0687dff into main Feb 5, 2026
5 checks passed
@yrh012 yrh012 deleted the fix/empty-param branch February 5, 2026 09:05
@github-actions github-actions bot locked and limited conversation to collaborators Feb 5, 2026
@domire8 domire8 linked an issue Feb 5, 2026 that may be closed by this pull request
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Empty parameters do not raise EmptyStateException in Python

3 participants