Skip to content
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

Fix Issue 2179 - Installing Playwright on Ubuntu 24 LTS and using Python 3.12 #2180

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

emilienbev
Copy link

Description

Upgrades Playwright's version to 1.48.0 which supports Ubuntu 24 LTS, and adds Python 3.12 minor version support to main.py.

Issue Fixes

Fixes Issue #2179

Checklist:

  • I am pushing changes to the develop branch
  • I am using the recommended development environment
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have formatted and linted my code using python-black and pylint
  • I have cleaned up unnecessary files
  • My changes generate no new warnings
  • My changes follow the existing code-style
  • My changes are relevant to the project

Any other information (e.g how to test the changes)

Running a fresh installation on Ubuntu 24.04 LTS will now work, and running the program functions correctly.

Copy link
Contributor

This pull request is stale as it has been open for 7 days with no activity. Remove stale label or comment, or this will be closed in 10 days.

@github-actions github-actions bot added the stale label Nov 11, 2024
@@ -20,8 +20,8 @@ jobs:
repo-token: ${{ secrets.GITHUB_TOKEN }}
stale-issue-message: 'This issue is stale because it has been open 7 days with no activity. Remove stale label or comment, or this will be closed in 10 days.'
close-issue-message: 'Issue closed due to being stale. Please reopen if issue persists in latest version.'
days-before-stale: 6
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, just rebase your pr so that these changes are not reverted

@JasonLovesDoggo JasonLovesDoggo added before next release Issue must be solved / PR must be merged before next release and removed stale labels Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
before next release Issue must be solved / PR must be merged before next release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants