Skip to content

Conversation

@noritaka1166
Copy link
Contributor

@noritaka1166 noritaka1166 commented Jul 9, 2025

User description

🔗 Related Issues

💥 What does this PR do?

Fix typos in dotnet

🔧 Implementation Notes

💡 Additional Considerations

🔄 Types of changes


PR Type

Other


Description

  • Fix spelling errors in comments and documentation

  • Correct typos in variable names and method descriptions

  • Update changelog entries with proper spelling


Changes diagram

flowchart LR
  A["Source Files"] --> B["Fix Spelling Errors"]
  B --> C["Update Documentation"]
  B --> D["Correct Comments"]
  B --> E["Fix Changelog"]
  C --> F["Improved Code Quality"]
  D --> F
  E --> F
Loading

Changes walkthrough 📝

Relevant files
Documentation
13 files
ChromiumOptions.cs
Fix "exeption" to "exception" in comments                               
+2/-2     
CommandResponseExtensions.cs
Fix "impelementing" to "implementing" in documentation     
+1/-1     
ConsoleApiCalledEventArgs.cs
Fix "time stanp" to "time stamp" in comments                         
+2/-2     
DevToolsSession.cs
Fix "wont" to "won't" in comment                                                 
+1/-1     
DriverService.cs
Fix "resposne" to "response" in comment                                   
+1/-1     
FirefoxExtension.cs
Fix "temporarly" and "extenion" typos in comments               
+2/-2     
GlobalSuppressions.cs
Fix "propert" to "property" in suppression message             
+1/-1     
PointerInputDevice.cs
Fix "proprties" to "properties" in parameter documentation
+1/-1     
ILogContext.cs
Fix "emmiting" to "emitting" in documentation                       
+1/-1     
ILogger.cs
Fix "emmiting" to "emitting" in documentation                       
+1/-1     
PinnedScript.cs
Fix "perview" to "preview" in comment                                       
+1/-1     
PrintOptions.cs
Fix "doucment" to "document" in property documentation     
+1/-1     
CHANGELOG
Fix multiple spelling errors throughout changelog entries
+30/-30 
Tests
5 files
CookieImplementationTest.cs
Fix "dalmation" to "dalmatian" and "Reenable" to "Re-enable"
+2/-2     
UnexpectedAlertBehaviorTest.cs
Fix "acknowldegement" to "acknowledgement" in test attribute
+1/-1     
VirtualAuthenticatorTest.cs
Fix "non-existant" to "non-existent" in test                         
+1/-1     
WindowSwitchingTest.cs
Fix "non-existant" to "non-existent" in test comment         
+1/-1     
LoadableComponentTests.cs
Fix "Excpected" to "Expected" in test messages                     
+2/-2     

Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
  • @selenium-ci selenium-ci added C-dotnet .NET Bindings B-support Issue or PR related to support classes labels Jul 9, 2025
    @selenium-ci
    Copy link
    Member

    Thank you, @noritaka1166 for this code suggestion.

    The support packages contain example code that many users find helpful, but they do not necessarily represent
    the best practices for using Selenium, and the Selenium team is not currently merging changes to them.

    We actively encourage people to add the wrapper and helper code that makes sense for them to their own frameworks.
    If you have any questions, please contact us

    @selenium-ci selenium-ci closed this Jul 9, 2025
    @qodo-code-review
    Copy link
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    🎫 Ticket compliance analysis ❌

    5678 - Not compliant

    Non-compliant requirements:

    • Fix ConnectFailure error when instantiating ChromeDriver
    • Address connection refused errors that occur after first instance
    • Resolve issue specific to Ubuntu 16.04.4 with Chrome 65.0.3325.181 and ChromeDriver 2.35

    1234 - Not compliant

    Non-compliant requirements:

    • Fix JavaScript execution in link href on click() for Firefox
    • Resolve regression from version 2.47.1 to 2.48.0/2.48.2
    • Ensure JavaScript alerts work properly with Firefox 42.0
    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    @qodo-code-review
    Copy link
    Contributor

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Labels

    B-support Issue or PR related to support classes C-dotnet .NET Bindings Review effort 1/5

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    2 participants