Skip to content

Conversation

@yanglbme
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings December 26, 2025 23:51
@idoocs idoocs added core team Issues or pull requests from core team js Issues or Pull requests relate to .js code md Issues or Pull requests relate to .md files ts Issues or Pull requests relate to .ts code labels Dec 26, 2025
Copy link

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

This PR updates TypeScript and JavaScript solutions for LeetCode problems to use a new PriorityQueue API instead of the deprecated MinPriorityQueue API. The changes standardize how priority queues are instantiated and accessed across multiple problem solutions.

Key Changes:

  • Replaced MinPriorityQueue with PriorityQueue using comparator functions
  • Updated queue access patterns to directly return values instead of wrapper objects with .element property
  • Applied changes consistently across both solution files and documentation (README files)

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated no comments.

Show a summary per file
File Description
solution/2500-2599/2599.Make the Prefix Sum Non-negative/Solution.ts Updated to use generic PriorityQueue<number>() and direct value access from dequeue()
solution/2500-2599/2599.Make the Prefix Sum Non-negative/README_EN.md Synchronized documentation code examples with the updated TypeScript solution
solution/2500-2599/2599.Make the Prefix Sum Non-negative/README.md Synchronized documentation code examples with the updated TypeScript solution
solution/2500-2599/2577.Minimum Time to Visit a Cell In a Grid/Solution.ts Replaced MinPriorityQueue with PriorityQueue<number[]> using comparator and updated dequeue access
solution/2500-2599/2577.Minimum Time to Visit a Cell In a Grid/Solution.js Applied corresponding JavaScript changes with non-generic PriorityQueue
solution/2500-2599/2577.Minimum Time to Visit a Cell In a Grid/README_EN.md Updated documentation including removal of trailing whitespace in CSS style block
solution/2500-2599/2577.Minimum Time to Visit a Cell In a Grid/README.md Synchronized documentation with updated solutions
solution/0300-0399/0347.Top K Frequent Elements/Solution.ts Changed to PriorityQueue<number[]> with tuple-based enqueue and array indexing for results
solution/0300-0399/0347.Top K Frequent Elements/README_EN.md Updated documentation to match solution changes
solution/0300-0399/0347.Top K Frequent Elements/README.md Updated documentation to match solution changes
solution/0200-0299/0295.Find Median from Data Stream/Solution.js Updated front() and dequeue() calls to access values directly without .element
solution/0200-0299/0295.Find Median from Data Stream/README_EN.md Synchronized documentation with JavaScript solution updates
solution/0200-0299/0295.Find Median from Data Stream/README.md Synchronized documentation with JavaScript solution updates

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

@yanglbme yanglbme merged commit 26feedb into main Dec 27, 2025
17 checks passed
@yanglbme yanglbme deleted the dev branch December 27, 2025 00:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core team Issues or pull requests from core team js Issues or Pull requests relate to .js code md Issues or Pull requests relate to .md files ts Issues or Pull requests relate to .ts code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants