inducer / inducer/codepy

All args for Toolchain subclasses now mandatory with release of v2025.1

Open
#54 3 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
57
Forks
25
Avg merge
13h 59m
Merged PRs (30d)
1

Description

With [this change](https://github.com/inducer/codepy/blame/3a0492ee2f29e325a28ea8aa049714a873d10649/codepy/toolchain.py#L55) (repeated for each class in the file), bundled in the most recent PyPi release, it appears that all args for instantiating subclasses of `Toolchain` are now mandatory which has forced us to fix our dependencies on a previous release and burn a new version.

I believe `@dataclass(frozen=True, kw_only=True)` would fix this issue.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.