Skip to content

Add string.Reverse()#3

Merged
Tyrrrz merged 3 commits intoprimefrom
copilot/add-string-reverse
Apr 14, 2026
Merged

Add string.Reverse()#3
Tyrrrz merged 3 commits intoprimefrom
copilot/add-string-reverse

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 12, 2026

Adds a Reverse() extension method on string that returns the characters in reverse order.

"hello".Reverse() // "olleh"

@Tyrrrz Tyrrrz added the enhancement New feature or request label Apr 12, 2026
@Tyrrrz Tyrrrz marked this pull request as ready for review April 12, 2026 12:11
Copilot AI review requested due to automatic review settings April 12, 2026 12:11
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.

Pull request overview

Adds a new Reverse() string extension to PowerKit to return characters in reverse order, along with corresponding unit tests.

Changes:

  • Added Reverse() to PowerKit.Extensions.StringExtensions.
  • Added Reverse_Test coverage in PowerKit.Tests.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
PowerKit/Extensions/StringExtensions.cs Introduces Reverse() extension that returns a reversed string.
PowerKit.Tests/StringExtensionsTests.cs Adds unit tests validating Reverse() behavior for typical and edge inputs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread PowerKit/Extensions/StringExtensions.cs Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@Tyrrrz
Copy link
Copy Markdown
Owner

Tyrrrz commented Apr 14, 2026

@copilot please update PolyShim to v2.9.0, I've added polyfills that should fix some of the issues you've outlined.

Agent-Logs-Url: https://github.com/Tyrrrz/PowerKit/sessions/374b0050-6171-449a-a049-1752b539134e

Co-authored-by: Tyrrrz <1935960+Tyrrrz@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 14, 2026

@copilot please update PolyShim to v2.9.0, I've added polyfills that should fix some of the issues you've outlined.

Done — updated PolyShim to v2.9.0 in commit dca3116. All 50 tests still pass.

Copilot AI requested a review from Tyrrrz April 14, 2026 18:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants