Skip to content

Conversation

@janskaar
Copy link
Contributor

@janskaar janskaar commented Feb 6, 2024

I somehow messed up the old branch, this is the test updated from the first code review. Also improved function / variable names in two other tests

@janskaar janskaar requested a review from heplesser February 6, 2024 14:05
@heplesser heplesser added S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation. I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) labels Feb 6, 2024
Copy link
Contributor

@heplesser heplesser left a comment

Choose a reason for hiding this comment

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

Much nicer now, just a few small comments left.

@janskaar janskaar requested a review from heplesser February 6, 2024 16:00
@heplesser heplesser merged commit 6a898bf into nest:master Feb 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants