Skip to content

[TASK] Drop magic method forwarding in OutputFormat#898

Merged
JakeQZ merged 1 commit intomainfrom
cleanup/avoid-forwarding
Mar 13, 2025
Merged

[TASK] Drop magic method forwarding in OutputFormat#898
JakeQZ merged 1 commit intomainfrom
cleanup/avoid-forwarding

Conversation

@oliverklee
Copy link
Collaborator

No description provided.

@coveralls
Copy link

coveralls commented Feb 10, 2025

Coverage Status

coverage: 55.84% (+0.1%) from 55.721%
when pulling 7e34b1d on cleanup/avoid-forwarding
into c226c04 on main.

@oliverklee

This comment was marked as outdated.

@oliverklee oliverklee force-pushed the cleanup/avoid-forwarding branch 17 times, most recently from 35d5c71 to adc2fd0 Compare February 16, 2025 21:19
@oliverklee oliverklee force-pushed the cleanup/avoid-forwarding branch 6 times, most recently from 880a39b to f7221ad Compare February 24, 2025 09:52
@oliverklee oliverklee force-pushed the cleanup/avoid-forwarding branch 9 times, most recently from 9c99531 to 535e12b Compare March 11, 2025 18:11
@oliverklee oliverklee force-pushed the cleanup/avoid-forwarding branch 10 times, most recently from 5eace31 to 7eb5e19 Compare March 12, 2025 17:56
@oliverklee oliverklee changed the title [TASK] Avoid magic method forwarding in OutputFormat [TASK] Drop magic method forwarding in OutputFormat Mar 12, 2025
@oliverklee oliverklee force-pushed the cleanup/avoid-forwarding branch from 7eb5e19 to 7ce2036 Compare March 12, 2025 18:02
@oliverklee oliverklee marked this pull request as ready for review March 12, 2025 18:11
@oliverklee oliverklee requested a review from JakeQZ March 12, 2025 18:11
Copy link
Collaborator

@JakeQZ JakeQZ left a comment

Choose a reason for hiding this comment

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

Should there be a changelog entry? There was one for the deprecation of this in #894.

@oliverklee oliverklee force-pushed the cleanup/avoid-forwarding branch 2 times, most recently from ea24e22 to 11ee02d Compare March 13, 2025 17:19
@oliverklee
Copy link
Collaborator Author

Sorry, I seem to have missed your comment. I've added a changelog entry now and repushed.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants