Fix linters and chrome selenium integration

[waynehamadi]

Background

Changes

PR Quality Checklist

• [X] I have run the following commands against my code to ensure it passes our linters:

  black . --exclude test.py
  isort .
  mypy .
  autoflake --remove-all-unused-imports --recursive --ignore-init-module-imports --ignore-pass-after-docstring --in-place agbenchmark

[github-actions[bot]]

PR Analysis

• 🎯 Main theme: The PR is focused on fixing linter issues and improving the integration of Chrome and Selenium.

• 📌 Type of PR: Bug fix

• ✨ Focused PR: Yes, all changes are related to the main theme of fixing linter issues and improving the integration of Chrome and Selenium.

  PR Feedback

• General suggestions: The PR seems to be well-structured and focused. However, it would be beneficial to add tests to ensure the changes work as expected and to prevent future regressions.

• 🤖 Code feedback:

  • relevant file: agbenchmark/utils/challenge.py suggestion: Consider using pathlib's built-in methods for path manipulations instead of converting the path to a string. This can make the code more readable and pythonic. [medium] relevant line: str(Path(self.CHALLENGE_LOCATION).parent),

  • relevant file: .github/workflows/ci.yml suggestion: It would be better to handle the case where the CHROMEDRIVER_VERSION or CHROME_VERSION is not found or the curl command fails. This can prevent potential failures in the CI pipeline. [important] relevant line:

How to use

To invoke the PR-Agent, add a comment using one of the following commands: /review [-i]: Request a review of your Pull Request. For an incremental review, which only considers changes since the last review, include the '-i' option. /describe: Modify the PR title and description based on the contents of the PR. /improve: Suggest improvements to the code in the PR. /ask \<QUESTION>: Pose a question about the PR. /update_changelog: Update the changelog based on the PR's contents.

To edit any configuration parameter from configuration.toml, add --config_path=new_value For example: /review --pr_reviewer.extra_instructions="focus on the file: ..." To list the possible configuration parameters, use the /config command.