Skip to content

chore: final clean up of Class._wrapFunction#5839

Open
hjpalpha wants to merge 4 commits intomainfrom
final-cleanup-of-class._wrapFunction
Open

chore: final clean up of Class._wrapFunction#5839
hjpalpha wants to merge 4 commits intomainfrom
final-cleanup-of-class._wrapFunction

Conversation

@hjpalpha
Copy link
Copy Markdown
Collaborator

@hjpalpha hjpalpha commented Apr 17, 2025

prereq:

Summary

Simplify Class._wrapFunction heavily after the prerequisites have been fulfilled.

How did you test this change?

dev

@hjpalpha
Copy link
Copy Markdown
Collaborator Author

failed tests likely due to need of prereqs

@hjpalpha hjpalpha linked an issue Apr 17, 2025 that may be closed by this pull request
19 tasks
@hjpalpha hjpalpha mentioned this pull request Apr 17, 2025
19 tasks
@Rathoz
Copy link
Copy Markdown
Collaborator

Rathoz commented Aug 19, 2025

Status?

@hjpalpha
Copy link
Copy Markdown
Collaborator Author

Status?

should work (tested as far as reasonable) and technically rfm
we wanted to do it when no big stuff is going on

@Rathoz Rathoz marked this pull request as ready for review March 5, 2026 16:21
@Rathoz Rathoz requested a review from a team as a code owner March 5, 2026 16:21
Copilot AI review requested due to automatic review settings March 5, 2026 16:21
@Rathoz Rathoz requested a review from a team as a code owner March 5, 2026 16:21
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Class.export cleanup

3 participants