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 install_vm.py on older versions of Python #10651

Merged
merged 1 commit into from
May 29, 2023

Conversation

jan-cerny
Copy link
Collaborator

We will remove the type hints that use collections. According to https://peps.python.org/pep-0585/, the "Type Hinting Generics In Standard Collections" were introduced in Python 3.9, but RHEL 8 and 7 use Python 3.6.

Fixes: #10650

We will remove the type hints that use collections.
According to https://peps.python.org/pep-0585/, the
"Type Hinting Generics In Standard Collections" were introduced
in Python 3.9, but RHEL 8 and 7 use Python 3.6.

Fixes: ComplianceAsCode#10650
@jan-cerny jan-cerny added bugfix Fixes to reported bugs. Test Suite Update in Test Suite. labels May 29, 2023
@jan-cerny jan-cerny added this to the 0.1.69 milestone May 29, 2023
@jan-cerny jan-cerny added the backported-into-stabilization PRs which were cherry-picked during stabilization process. label May 29, 2023
@github-actions
Copy link

Start a new ephemeral environment with changes proposed in this pull request:

Fedora Environment
Open in Gitpod

Oracle Linux 8 Environment
Open in Gitpod

Copy link
Collaborator

@vojtapolasek vojtapolasek left a comment

Choose a reason for hiding this comment

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

LGTM

@codeclimate
Copy link

codeclimate bot commented May 29, 2023

Code Climate has analyzed commit c6c31d2 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 52.5% (0.0% change).

View more on Code Climate.

@mildas
Copy link
Contributor

mildas commented May 29, 2023

LGTM

@mildas mildas self-assigned this May 29, 2023
@mildas mildas merged commit 47955e5 into ComplianceAsCode:master May 29, 2023
28 checks passed
jan-cerny added a commit to jan-cerny/scap-security-guide that referenced this pull request May 30, 2023
We will remove the type hints that use collections.
According to https://peps.python.org/pep-0585/, the
"Type Hinting Generics In Standard Collections" were introduced
in Python 3.9, but RHEL 8 and 7 use Python 3.6.

This is a backport of ComplianceAsCode#10651.

Fixes: ComplianceAsCode#10650
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backported-into-stabilization PRs which were cherry-picked during stabilization process. bugfix Fixes to reported bugs. Test Suite Update in Test Suite.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

install_vm.py script: TypeError: 'type' object is not subscriptable
3 participants