Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Sign up
Appearance settings

Update "First Script" documentation with C# example from .NET README #2478

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.

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

Open
rushixdd wants to merge 1 commit into SeleniumHQ:trunk
base: trunk
Choose a base branch
Loading
from rushixdd:v-rd/dotnet-readme

Conversation

@rushixdd
Copy link

@rushixdd rushixdd commented Sep 11, 2025 •
edited by qodo-merge-pro bot
Loading

User description

Updated multiple language versions of the "First Script" documentation to include the C# example from the newly added .NET README. The following languages were updated:

English (first_script.en.md)

Japanese (first_script.ja.md)

Portuguese (first_script.pt-br.md)

Simplified Chinese (first_script.zh-cn.md)

These updates include a new code block reference pointing to the appropriate section in the .NET README file.

PR Type

Documentation

Description

  • Added comprehensive .NET README with setup and testing instructions

  • Updated C# code references in multilingual documentation

  • Replaced placeholder badges with actual code examples

  • Enhanced documentation consistency across four languages

Diagram Walkthrough

flowchart LR
 A["New .NET README"] --> B["Setup Instructions"]
 A --> C["Test Commands"]
 D["Documentation Files"] --> E["Updated C# References"]
 E --> F["English/Japanese/Portuguese/Chinese"]
Loading

File Walkthrough
Relevant files
Documentation
README.md
New .NET README with setup instructions

examples/dotnet/README.md

  • Created comprehensive .NET setup and testing guide
  • Added prerequisites, installation, and CLI commands
  • Included specific test execution examples
+61/-0
first_script.en.md
Updated C# code reference in English docs

website_and_docs/content/documentation/webdriver/getting_started/first_script.en.md

  • Replaced {{< badge-code >}} with actual code reference
  • Added link to .NET README line 55
+1/-1
first_script.ja.md
Updated C# code reference in Japanese docs

website_and_docs/content/documentation/webdriver/getting_started/first_script.ja.md

  • Replaced {{< badge-code >}} with actual code reference
  • Added link to .NET README line 55
+1/-1
first_script.pt-br.md
Updated C# code reference in Portuguese docs

website_and_docs/content/documentation/webdriver/getting_started/first_script.pt-br.md

  • Replaced {{< badge-code >}} with actual code reference
  • Added link to .NET README line 55
+1/-1
first_script.zh-cn.md
Updated C# code reference in Chinese docs

website_and_docs/content/documentation/webdriver/getting_started/first_script.zh-cn.md

  • Replaced {{< badge-code >}} with actual code reference
  • Added link to .NET README line 55
+1/-1

Copy link

netlify bot commented Sep 11, 2025 •
edited
Loading

👷 Deploy request for selenium-dev pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 34d5468

Copy link

CLAassistant commented Sep 11, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

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

Incorrect Reference

The CSharp tab references /examples/dotnet/README.md#L55, which appears to be a CLI test filter command rather than the "First Script" C# code sample. Ensure the link points to an actual C# example; this applies to all translated pages updated in this PR.

 
{{< gh-codeblock path="/examples/dotnet/README.md#L55" >}}
{{< /tab >}}
Missing Code Example

The new .NET README provides setup and test commands but no C# "First Script" code snippet to reference. Consider adding a dedicated code block (with a stable anchor or section) for the first script so documentation tabs can link to it reliably instead of a fragile line number.

 
### Run a Specific Test Method
To run a specific test method using its fully qualified name:
```bash
dotnet test --filter "FullyQualifiedName=Namespace.ClassName.MethodName"

For example:

dotnet test --filter "FullyQualifiedName=SeleniumDocs.GettingStarted.UsingSeleniumTest.EightComponents"

You can also filter by test name only (if it's unique):

dotnet test --filter "Name=EightComponents"

</details>
</td></tr>
</table>

Copy link
Contributor

qodo-merge-pro bot commented Sep 11, 2025 •
edited
Loading

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion Impact
Possible issue
Fix incorrect C# snippet reference

This shortcode points to a bash command in the README, not a C# "first script"
example, so the C# tab will render incorrect content. Revert to the placeholder
until a proper C# code snippet is available or update it to reference an actual
C# example file.

website_and_docs/content/documentation/webdriver/getting_started/first_script.en.md [239] 

-{{< gh-codeblock path="/examples/dotnet/README.md#L55" >}}
+{{< badge-code >}}
  • Apply / Chat
Suggestion importance[1-10]: 8

__

Why: The suggestion correctly points out that the linked code points to a shell command in a README.md file, not a C# code example, which is misleading for users on a "first script" page.

Medium
Correct C# tab content source

The .NET include targets a README line with a CLI command, not a C# code sample,
causing the C# tab to display the wrong content. Restore the placeholder until
the correct C# snippet path is available.

website_and_docs/content/documentation/webdriver/getting_started/first_script.pt-br.md [241] 

-{{< gh-codeblock path="/examples/dotnet/README.md#L55" >}}
+{{< badge-code >}}
  • Apply / Chat
Suggestion importance[1-10]: 8

__

Why: The suggestion correctly points out that the linked code points to a shell command in a README.md file, not a C# code example, which is misleading for users on a "first script" page.

Medium
Replace wrong README reference

Referencing a README line shows a bash command rather than the intended C#
example, breaking the "First Script" content. Switch back to the placeholder
until a stable C# code include is identified.

website_and_docs/content/documentation/webdriver/getting_started/first_script.zh-cn.md [254] 

-{{< gh-codeblock path="/examples/dotnet/README.md#L55" >}}
+{{< badge-code >}}
  • Apply / Chat
Suggestion importance[1-10]: 8

__

Why: The suggestion correctly points out that the linked code points to a shell command in a README.md file, not a C# code example, which is misleading for users on a "first script" page.

Medium
  • Update 

...p and running Selenium tests using C#.
Updated the "First Script" documentation in multiple languages to include references to the new .NET README with C# example code blocks:
English (first_script.en.md)
Japanese (first_script.ja.md)
Portuguese (first_script.pt-br.md)
Simplified Chinese (first_script.zh-cn.md)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /