Skip to content

Conversation

@cgoldberg
Copy link
Member

@cgoldberg cgoldberg commented Aug 15, 2025

User description

💥 What does this PR do?

Fix whitespace that broke the build.

🔄 Types of changes

  • Bug fix (backwards compatible)

PR Type

Bug fix


Description

  • Remove trailing whitespace from NUMPAD0 constant

  • Remove empty line after NUMPAD9 constant


File Walkthrough

Relevant files
Formatting
keys.py
Fix whitespace in Keys constants                                                 

py/selenium/webdriver/common/keys.py

  • Removed trailing whitespace from NUMPAD0 constant definition
  • Removed extra blank line after NUMPAD9 constant
+2/-2     

@cgoldberg cgoldberg merged commit ae953fc into SeleniumHQ:trunk Aug 15, 2025
4 of 5 checks passed
@selenium-ci selenium-ci added the C-py Python Bindings label Aug 15, 2025
@cgoldberg cgoldberg deleted the py-fix-formatting branch August 15, 2025 20:13
@qodo-merge-pro
Copy link
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Consistency

Confirm other constants around the edited lines have no trailing whitespace or unintended blank lines to avoid future formatting-induced build failures.

NUMPAD0 = "\ue01a"
NUMPAD1 = "\ue01b"
NUMPAD2 = "\ue01c"
NUMPAD3 = "\ue01d"
NUMPAD4 = "\ue01e"
NUMPAD5 = "\ue01f"
NUMPAD6 = "\ue020"
NUMPAD7 = "\ue021"
NUMPAD8 = "\ue022"
NUMPAD9 = "\ue023"

MULTIPLY = "\ue024"
ADD = "\ue025"

@qodo-merge-pro
Copy link
Contributor

PR Code Suggestions ✨

No code suggestions found for the PR.

This was referenced Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants