SeleniumHQ / seleniumhq.github.io

Official Selenium website and documentation
https://selenium.dev
Apache License 2.0
1.1k stars 1.29k forks source link

Update the service lines on IE explorer examples for ruby #1761

Closed aguspe closed 3 months ago

aguspe commented 3 months ago

User description

Description

On my previous PR #1748 I updated the examples but I did not update the lines for the service examples, so now it looks like there are none

Screenshot 2024-06-19 at 13 40 44

I also updated the format of the previous added IE examples

Motivation and Context

It's important that all of the Selenium users have access to high-quality examples to increase adoption and facilitate usage

Types of changes

Checklist


PR Type

Documentation


Description


Changes walkthrough πŸ“

Relevant files
Documentation
internet_explorer.en.md
Update Ruby code example paths in IE documentation             

website_and_docs/content/documentation/webdriver/browsers/internet_explorer.en.md
  • Updated Ruby code example paths to point to single lines instead of
    ranges.
  • Ensured consistency in code example references.
  • +11/-11 
    internet_explorer.ja.md
    Update Ruby code example paths in IE documentation (Japanese)

    website_and_docs/content/documentation/webdriver/browsers/internet_explorer.ja.md
  • Updated Ruby code example paths to point to single lines instead of
    ranges.
  • Ensured consistency in code example references.
  • +11/-11 
    internet_explorer.pt-br.md
    Update Ruby code example paths in IE documentation (Portuguese)

    website_and_docs/content/documentation/webdriver/browsers/internet_explorer.pt-br.md
  • Updated Ruby code example paths to point to single lines instead of
    ranges.
  • Ensured consistency in code example references.
  • +11/-11 
    internet_explorer.zh-cn.md
    Update Ruby code example paths in IE documentation (Chinese)

    website_and_docs/content/documentation/webdriver/browsers/internet_explorer.zh-cn.md
  • Updated Ruby code example paths to point to single lines instead of
    ranges.
  • Ensured consistency in code example references.
  • +11/-11 

    πŸ’‘ PR-Agent usage: Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    netlify[bot] commented 3 months ago

    Deploy request for selenium-dev pending review.

    Visit the deploys page to approve it

    Name Link
    Latest commit 6f49c1c65d87d291f331bc2a9b500ab2d666b891
    codiumai-pr-agent-pro[bot] commented 3 months ago

    PR Reviewer Guide πŸ”

    ⏱️ Estimated effort to review [1-5] 2
    πŸ§ͺ Relevant tests No
    πŸ”’ Security concerns No
    ⚑ Key issues to review None
    codiumai-pr-agent-pro[bot] commented 3 months ago

    PR Code Suggestions ✨

    CategorySuggestion                                                                                                                                    Score
    Possible issue
    Verify and correct the gh-codeblock paths to ensure they reference the correct lines in the Ruby example files ___ **Ensure that the new gh-codeblock paths are correct and point to the intended lines in the
    Ruby example files. The previous paths included a range (e.g., #L29-L30), which might
    indicate that more than one line is relevant. Verify that the single line references
    (e.g., #L29) are accurate and complete.** [website_and_docs/content/documentation/webdriver/browsers/internet_explorer.pt-br.md [118]](https://github.com/SeleniumHQ/seleniumhq.github.io/pull/1761/files#diff-fe2d01308c167a177a9d3bbb5ab5d10e71e644ca3fe386c5936fd9de12346b03R118-R118) ```diff -{{< gh-codeblock path="/examples/ruby/spec/browsers/internet_explorer_spec.rb#L29" >}} +{{< gh-codeblock path="/examples/ruby/spec/browsers/internet_explorer_spec.rb#L29-L30" >}} ``` - [ ] **Apply this suggestion**
    Suggestion importance[1-10]: 7 Why: The suggestion correctly identifies a potential issue with the change from a range of lines to a single line in the `gh-codeblock` paths. This could lead to incomplete references, which is important for accurate documentation.
    7
    Maintainability
    Maintain consistency in gh-codeblock paths across different language documentation ___ **Consider maintaining consistency in the gh-codeblock paths across all languages. If the
    paths in the Portuguese documentation are updated to single lines, ensure the same is done
    in the Chinese documentation for consistency.** [website_and_docs/content/documentation/webdriver/browsers/internet_explorer.pt-br.md [118]](https://github.com/SeleniumHQ/seleniumhq.github.io/pull/1761/files#diff-fe2d01308c167a177a9d3bbb5ab5d10e71e644ca3fe386c5936fd9de12346b03R118-R118) ```diff -{{< gh-codeblock path="/examples/ruby/spec/browsers/internet_explorer_spec.rb#L29" >}} +{{< gh-codeblock path="/examples/ruby/spec/browsers/internet_explorer_spec.rb#L29-L30" >}} ``` - [ ] **Apply this suggestion**
    Suggestion importance[1-10]: 5 Why: This suggestion is valid as it promotes consistency across documentation translations, which is beneficial for maintainability and user experience. However, it's less critical than ensuring the accuracy of the documentation paths.
    5