Skip to content

Add optimization to GetRawData #653

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
Merged
Shane32 merged 1 commit into from
Oct 7, 2025
Merged

Add optimization to GetRawData #653

Shane32 merged 1 commit into from
Oct 7, 2025

Conversation

@Shane32
Copy link
Owner

@Shane32 Shane32 commented Oct 7, 2025

No description provided.

@Shane32 Shane32 added this to the 1.7.0 milestone Oct 7, 2025
@Shane32 Shane32 requested a review from gfoidl October 7, 2025 21:28
@Shane32 Shane32 self-assigned this Oct 7, 2025
@Shane32 Shane32 added the performance Performance related enhancements or benchmarks label Oct 7, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 7, 2025

Warning

Rate limit exceeded

@Shane32 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 23 minutes and 31 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 317c587 and d6ebf74.

📒 Files selected for processing (1)
  • QRCoder/QRCodeData.cs (1 hunks)
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch getrawdata_opt

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Shane32 Shane32 merged commit f1ddb37 into master Oct 7, 2025
8 checks passed
@Shane32 Shane32 deleted the getrawdata_opt branch October 7, 2025 22:26
This was referenced Oct 9, 2025
Merged
Closed
Merged
Open
Open
Merged
Merged
@dependabot dependabot bot mentioned this pull request Oct 27, 2025
Merged
@dependabot dependabot bot mentioned this pull request Nov 14, 2025
Merged
This was referenced Nov 25, 2025
Merged
Open
This was referenced Dec 5, 2025
Closed
Open
Open
@dependabot dependabot bot mentioned this pull request Dec 28, 2025
Closed
@dependabot dependabot bot mentioned this pull request Jan 7, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gfoidl gfoidl gfoidl approved these changes

Assignees

@Shane32 Shane32

Labels

performance Performance related enhancements or benchmarks

Projects

None yet

Milestone

1.7.0

Development

Successfully merging this pull request may close these issues.

3 participants

@Shane32 @gfoidl