Skip to content

[WIP] Update pre-commit Ruff version and fix errors - #94

Open
EZoni wants to merge 3 commits into
pals-project:mainfrom
EZoni:precommit_update
Open

EZoni wants to merge 3 commits into
pals-project:mainfrom
EZoni:precommit_update

Conversation

@EZoni

@EZoni EZoni commented Oct 12, 2026

Copy link
Copy Markdown
Member

Updated the Ruff version in the pre-commit hook: astral-sh/ruff-pre-commit@v0.15.12...v0.17.0.

@EZoni EZoni added the CI/CD label Oct 12, 2026
@EZoni EZoni changed the title Update Ruff version in pre-commit hook [WIP] Update pre-commit Ruff version and fix errors Oct 12, 2026
Comment on lines -12 to +14
location: str = "" # Where reference parameters are evaluated
location: Literal[
"UPSTREAM_END", "DOWNSTREAM_END"
] # TODO: undefined default in PALS?
# TODO: not defined in PALS
location: Literal["UPSTREAM_END", "DOWNSTREAM_END"] | None = None

@EZoni EZoni Oct 12, 2026 •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Incidentally, it seems like this (i.e., having location defined twice) was a bug. I think any dumped lattice that previously contained location: "" will now contain location: null. Note that this is still undefined in PALS.

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.

1 participant