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

  • Cleanup (formatting, renaming)
  • Bug fix (backwards compatible)
  • New feature (non-breaking change which adds functionality and tests!)
  • Breaking change (fix or feature that would cause existing functionality to change)

PR Type

Other


Description

  • Fix spelling errors in .NET codebase comments and documentation

  • Correct typos in GlobalSuppressions.cs justification messages

  • Update CHANGELOG and test files with proper spelling


Changes diagram

flowchart LR
  A["Typo Detection"] --> B["GlobalSuppressions.cs"]
  A --> C["Source Files"]
  A --> D["Test Files"]
  A --> E["CHANGELOG"]
  B --> F["Fixed 'execptions' to 'exceptions'"]
  C --> G["Fixed various spelling errors"]
  D --> H["Corrected test comments"]
  E --> I["Updated historical entries"]
Loading

Changes walkthrough 📝

Relevant files
Formatting
20 files
GlobalSuppressions.cs
Fix 'execptions' to 'exceptions' in justifications             
+38/-38 
ChromiumOptions.cs
Fix 'exeption' to 'exception' in comments                               
+2/-2     
CommandResponseExtensions.cs
Fix 'impelementing' to 'implementing' in comment                 
+1/-1     
ConsoleApiCalledEventArgs.cs
Fix 'stanp' to 'stamp' in documentation                                   
+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                                       
+2/-2     
GlobalSuppressions.cs
Fix 'propert' to 'property' in justification                         
+1/-1     
PointerInputDevice.cs
Fix 'proprties' to 'properties' in comment                             
+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 documentation                       
+1/-1     
CookieImplementationTest.cs
Fix 'dalmation' to 'dalmatian' and 'Reenable' to 'Re-enable'
+2/-2     
UnexpectedAlertBehaviorTest.cs
Fix 'acknowldegement' to 'acknowledgement' in comment       
+1/-1     
VirtualAuthenticatorTest.cs
Fix 'non-existant' to 'non-existent' in test                         
+1/-1     
WindowSwitchingTest.cs
Fix 'non-existant' to 'non-existent' in comment                   
+1/-1     
LoadableComponentTests.cs
Fix 'Excpected' to 'Expected' in test message                       
+2/-2     
CHANGELOG
Fix multiple typos throughout changelog history                   
+30/-30 
Selenium.WebDriver.Support.csproj
Fix 'utilites' to 'utilities' in description                         
+1/-1     

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

    @qodo-code-review
    Copy link
    Contributor

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    🎫 Ticket compliance analysis ❌

    1234 - Not compliant

    Non-compliant requirements:

    • Fix issue where JavaScript in link's href is not triggered on click() in version 2.48
    • Ensure compatibility with Firefox 42.0 32bit on 64bit machine
    • Restore functionality that worked in version 2.47.1

    5678 - Not compliant

    Non-compliant requirements:

    • Fix ChromeDriver ConnectFailure error on Ubuntu 16.04.4
    • Resolve connection refused errors after first ChromeDriver instance
    • Support Chrome 65.0.3325.181 with ChromeDriver version 2.35
    • Fix issue with Selenium Version 3.9.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