Remove Python 2 CI job (#3727)

The Python 2 and 3 jobs run essentially the same thing twice. I asked for
a sign from implementations if any of them were relying on Python 2 for
their test262 runners, but didn't hear of any that were.

The Python tools are changed so rarely that I think it's OK to drop this
CI job. If a future change breaks Python 2 for someone, we can accept PRs
to fix it on a best-effort basis, or reinstate this CI job if it's really
necessary.

Closes: #3480

Co-authored-by: Richard Gibson <richard.gibson@gmail.com>
Co-authored-by: Ms2ger <Ms2ger@gmail.com>
This commit is contained in:
Philip Chimento 2022-11-30 07:43:23 -08:00 committed by GitHub
parent 39302ea3c1
commit 9b5c38d109
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 28 deletions

View File

@ -37,9 +37,9 @@ execution_steps: &execution_steps
- run: hostPath=$HOME/.esvu/bin/$hostPath npm run ci
jobs:
"Test262: verify tools; build & lint tests (Python 2)":
"Test262: verify tools; build & lint tests":
docker:
- image: cimg/python:2.7
- image: cimg/python:3.7.4
working_directory: ~/test262
steps:
- checkout
@ -64,30 +64,6 @@ jobs:
# - run:
# name: "Run deploy"
# command: ./tools/scripts/deploy.sh
"Test262: verify tools; build & lint tests (Python 3)":
docker:
- image: cimg/python:3.7.4
working_directory: ~/test262
steps:
- checkout
- run:
name: "Install requirements for generation tool"
command: python -m pip install --user --requirement tools/generation/requirements.txt
- run:
name: "Install requirements for lint tool"
command: python -m pip install --user --requirement tools/lint/requirements.txt
- run:
name: "Test the generation tool"
command: ./tools/generation/test/run.py
- run:
name: "Test the lint tool"
command: ./tools/lint/test/run.py
- run:
name: "Build tests; check for new changes"
command: ./tools/scripts/ci_build.sh
- run:
name: "Lint tests"
command: ./tools/scripts/ci_lint.sh
"V8: New or modified tests execution":
docker:
- image: *node_latest
@ -170,8 +146,7 @@ workflows:
version: 2
Tools:
jobs:
- "Test262: verify tools; build & lint tests (Python 2)"
- "Test262: verify tools; build & lint tests (Python 3)"
- "Test262: verify tools; build & lint tests"
Tests execution:
jobs:
- "ChakraCore: New or modified tests execution"