Skip to content

[WIP] Address feedback on 'Encode hubs' pull request #1801

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
Copilot wants to merge 1 commit into from
Closed

[WIP] Address feedback on 'Encode hubs' pull request #1801

Copilot wants to merge 1 commit into from

Conversation

Copy link

Copilot AI commented Jan 3, 2026 •
edited by jrobinso
Loading

  • Understand the feedback: keySet() should return original keys, not lowercase
  • Modify CaseInsensitiveMap to store both original and lowercase keys
  • Update put() method to store original key mapping
  • Update keySet() method to return original keys
  • Update entrySet() method to return entries with original keys
  • Create tests to verify the fix
  • Run tests to validate changes
  • Run code review
  • Run security scan

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI mentioned this pull request Jan 3, 2026
Merged
@jrobinso jrobinso marked this pull request as ready for review January 3, 2026 07:11
@jrobinso jrobinso closed this Jan 3, 2026
Copilot AI requested a review from jrobinso January 3, 2026 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jrobinso jrobinso Awaiting requested review from jrobinso

Assignees

@jrobinso jrobinso

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jrobinso