Skip to content

Conversation

Starbuck5
Copy link
Member

I noticed this on the deployed site, when I went to click on the link.

@Starbuck5 Starbuck5 requested a review from a team as a code owner October 9, 2025 07:15
@Starbuck5 Starbuck5 added docs Surface pygame.Surface labels Oct 9, 2025
Copy link
Contributor

coderabbitai bot commented Oct 9, 2025

📝 Walkthrough

Walkthrough

Docstring link updated in a stub file: premul_alpha method’s tutorial URL changed to a new relative path. No code, signature, or behavior changes.

Changes

Cohort / File(s) Summary of Changes
Docstring link update
buildconfig/stubs/pygame/surface.pyi
Updated premul_alpha docstring link from "tutorials/en/premultiplied-alpha" to "../tutorials/en/premultiplied-alpha.html". No signature or logic changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title directly reflects the core change of updating the tutorial link for the premul_alpha method and is both concise and specific.
Description Check ✅ Passed The description explains that the link was broken on the deployed site and provides the motivation for fixing it, directly relating to the changeset.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between abe99c2 and 2ca539f.

📒 Files selected for processing (1)
  • buildconfig/stubs/pygame/surface.pyi (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (20)
  • GitHub Check: build (ubuntu-22.04)
  • GitHub Check: Debian (Bookworm - 12) [ppc64le]
  • GitHub Check: Debian (Bookworm - 12) [armv6]
  • GitHub Check: Debian (Bookworm - 12) [armv7]
  • GitHub Check: Debian (Bookworm - 12) [s390x]
  • GitHub Check: aarch64
  • GitHub Check: x86_64
  • GitHub Check: i686
  • GitHub Check: dev-check
  • GitHub Check: x86
  • GitHub Check: build (macos-14)
  • GitHub Check: debug_coverage (ubuntu-24.04, 3.9.23)
  • GitHub Check: debug_coverage (ubuntu-24.04, 3.14.0rc1)
  • GitHub Check: AMD64
  • GitHub Check: build (windows-latest)
  • GitHub Check: debug_coverage (ubuntu-24.04, 3.13.5)
  • GitHub Check: build (ubuntu-24.04)
  • GitHub Check: msys2 (ucrt64, ucrt-x86_64)
  • GitHub Check: msys2 (clang64, clang-x86_64)
  • GitHub Check: msys2 (mingw64, x86_64)
🔇 Additional comments (1)
buildconfig/stubs/pygame/surface.pyi (1)

1006-1006: Link update looks correct.

The relative path and .html suffix should resolve properly in the generated docs.


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.

Copy link
Member

@ankith26 ankith26 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@ankith26 ankith26 added this to the 2.5.6 milestone Oct 9, 2025
@ankith26 ankith26 merged commit 3528a9f into pygame-community:main Oct 9, 2025
28 checks passed
@Starbuck5 Starbuck5 deleted the fix-tutorial-link branch October 10, 2025 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Surface pygame.Surface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants