Skip to content

Apply latest Copier template updates#247

Open
AndrewSazonov wants to merge 8 commits intodevelopfrom
new-templates
Open

Apply latest Copier template updates#247
AndrewSazonov wants to merge 8 commits intodevelopfrom
new-templates

Conversation

@AndrewSazonov
Copy link
Copy Markdown
Member

This PR syncs the project with the latest Copier templates. Main changes include replacing deprecated Node.js 20 usage, migrating docstrings from Google style to NumPy style, and applying related template-driven configuration updates.

@AndrewSazonov AndrewSazonov added the [scope] maintenance Code/tooling cleanup, no feature or bugfix (major.minor.PATCH) label May 7, 2026
Copy link
Copy Markdown
Member

@henrikjacobsenfys henrikjacobsenfys left a comment

Choose a reason for hiding this comment

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

I looked at a handful of randomly selected files, and they looked fine.

Copy link
Copy Markdown
Member

@rozyczko rozyczko left a comment

Choose a reason for hiding this comment

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

I spent way too much time on this PR :)

Rechecked actions version bump correctness,
Assured client-id is the new version of app-ad in create-github-app-token action
Checked dependency move in pyproject.toml
Got worried about manually copying trstringer/require-label-prefix action
Very useful addition of nodefaults to pixi channels
etc.

One potential issue I have is - do we really, REALLY, need to add the useless docstrings to all the methods?

def __init__(self):
    """Init function."""
def __call__(self, *args, **kwargs):
    """Call function."""

Instead of adding junk, why not just # noqa: DOC comment for those trivial dunder methods?

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

Labels

[scope] maintenance Code/tooling cleanup, no feature or bugfix (major.minor.PATCH)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants