Skip to content

add Navin and Swastik to selenium committers #2331

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

Merged
merged 1 commit into from
Jun 4, 2025

Conversation

navin772
Copy link
Member

@navin772 navin772 commented Jun 4, 2025

User description

Thanks for contributing to the Selenium site and documentation!
A PR well described will help maintainers to review and merge it quickly

Before submitting your PR, please check our contributing guidelines.
Avoid large PRs, and help reviewers by making them as simple and short as possible.

Description

Adds Navin Chandra (navin772) and Swastik Baranwal (Delta456) to committers.

Motivation and Context

Types of changes

  • Change to the site (I have double-checked the Netlify deployment, and my changes look good)

Checklist

  • I have read the contributing document.
  • I have used hugo to render the site/docs locally and I am sure it works.

PR Type

Documentation


Description

  • Added Navin Chandra and Swastik Baranwal to Selenium committers list

  • Updated project structure documentation to reflect new committers


Changes walkthrough 📝

Relevant files
Documentation
_index.html
Add Navin Chandra and Swastik Baranwal to committers list

website_and_docs/content/project/structure/_index.html

  • Added Swastik Baranwal (Delta456) to committers
  • Added Navin Chandra (navin772) to committers
  • Updated the list of Selenium committers in documentation
  • +2/-0     

    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.
  • Copy link

    netlify bot commented Jun 4, 2025

    Deploy Preview for selenium-dev ready!

    Name Link
    🔨 Latest commit f267923
    🔍 Latest deploy log https://app.netlify.com/projects/selenium-dev/deploys/6840862190f24400080a7b11
    😎 Deploy Preview https://deploy-preview-2331--selenium-dev.netlify.app
    📱 Preview on mobile
    Toggle QR Code...

    QR Code

    Use your smartphone camera to open QR code link.

    To edit notification comments on pull requests, go to your Netlify project configuration.

    Copy link
    Contributor

    qodo-merge-pro bot commented Jun 4, 2025

    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
    ⚡ No major issues detected

    Copy link
    Contributor

    qodo-merge-pro bot commented Jun 4, 2025

    PR Code Suggestions ✨

    Explore these optional code suggestions:

    CategorySuggestion                                                                                                                                    Impact
    General
    Fix inconsistent indentation

    Fix the indentation inconsistency by using spaces instead of tabs for the new
    entry. The rest of the file uses spaces for indentation, but the new entry uses
    a tab which creates inconsistent formatting.

    website_and_docs/content/project/structure/_index.html [68-70]

         {{< gh-user "https://api.github.com/users/ddavison" >}}
    -+	{{< gh-user "https://api.github.com/users/Delta456" >}}
    +    {{< gh-user "https://api.github.com/users/Delta456" >}}
         {{< gh-user "https://api.github.com/users/dfabulich" >}}
    • Apply / Chat
    Suggestion importance[1-10]: 4

    __

    Why: The suggestion correctly identifies an indentation inconsistency where the new Delta456 entry uses a tab character while surrounding lines use spaces. This is a valid formatting issue that improves code consistency.

    Low
    • More

    @harsha509 harsha509 merged commit 386706d into SeleniumHQ:trunk Jun 4, 2025
    6 checks passed
    @navin772 navin772 deleted the patch-1 branch June 4, 2025 18:16
    selenium-ci added a commit that referenced this pull request Jun 4, 2025
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    2 participants