Skip to content

Conversation

@happz
Copy link
Contributor

@happz happz commented Dec 10, 2025

We collected several typos and some interesting items were not printed out. Polished that a bit, hopefully with attrs we will move this closer to the fields themselves.

Pull Request Checklist

  • implement the feature

We collected several typos and some interesting items were not printed
out. Polished that a bit, hopefully with `attrs` we will move this
closer to the fields themselves.
@happz happz added this to planning Dec 10, 2025
@happz happz added step | provision Stuff related to the provision step code | logging Changes related to debug logging ci | full test Pull request is ready for the full test execution labels Dec 10, 2025
@github-project-automation github-project-automation bot moved this to backlog in planning Dec 10, 2025
@happz happz moved this from backlog to review in planning Dec 10, 2025
Copy link
Contributor

@LecrisUT LecrisUT left a comment

Choose a reason for hiding this comment

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

Apart from the duplication reported by graphite, lgtm

@LecrisUT LecrisUT added the review | trivial Very easy for review, even for beginners, so don't be afraid to have a look! :-) label Dec 11, 2025
Co-authored-by: graphite-app[bot] <96075541+graphite-app[bot]@users.noreply.github.com>
@happz happz moved this from review to merge in planning Jan 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci | full test Pull request is ready for the full test execution code | logging Changes related to debug logging review | trivial Very easy for review, even for beginners, so don't be afraid to have a look! :-) step | provision Stuff related to the provision step

Projects

Status: merge

Development

Successfully merging this pull request may close these issues.

4 participants