Skip to content

Conversation

@felixfontein
Copy link
Collaborator

SUMMARY

Will likely merge this on ~Saturday.

For that, I will:

  1. Recreate the first commit from current state of main;
  2. Update second commit so first commit is mentioned in .git-blame-ignore-revs;
  3. Merge with rebase so that two commits end up in main.

This PR mainly shows how the result will look like.

ISSUE TYPE
  • Test Pull Request
COMPONENT NAME

all Python code

@felixfontein
Copy link
Collaborator Author

bot_skip

@felixfontein felixfontein added the breaking_change This PR contains a breaking change that MUST NOT be backported label Oct 27, 2025
@felixfontein felixfontein requested a review from russoz October 27, 2025 20:03
Copy link
Collaborator

@russoz russoz left a comment

Choose a reason for hiding this comment

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

Man, I will just trust the code. No way I can review 1000+ files before Sunday and keep my marriage at the same time.

@felixfontein
Copy link
Collaborator Author

Man, I will just trust the code. No way I can review 1000+ files before Sunday and keep my marriage at the same time.

I didn't expect you (or anyone else) to review everything :D (Also some things will change when I have to re-create the first commit.) The 'interesting' part is the second commit: ed71c3e

@felixfontein
Copy link
Collaborator Author

(Some manual post-processing is needed, but I will only do that when it's clear I don't have to re-do the work...)

@felixfontein felixfontein marked this pull request as ready for review November 1, 2025 12:31
@felixfontein felixfontein changed the title [WIP] Reformat code with 'ruff format' Reformat code with 'ruff format' Nov 1, 2025
@felixfontein
Copy link
Collaborator Author

The CI error is unrelated.

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

Labels

breaking_change This PR contains a breaking change that MUST NOT be backported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants