Skip to content

Commit

Permalink
update ci.yml to directly install py27 on windows runner (#197)
Browse files Browse the repository at this point in the history
* update ci.yml to use ubuntu instead of windows for py27

* change ubuntu 20.04 to lates

* change ubuntu to 22.04

* testing

* testing2

* test3

* testing to see if codecov runs on all tests

* remove sudo

* add exit when there's failing tests

* trying to exit when there's error

* testing exit when tests fails

* fail pipeline when there's failing test

* trying to fail th pipepline

* fail pipeline

* use condition to look into err2.txt

* test if py3 will fail

* add checking for failing in py3

* test if py3 failed or not

* all else

* use if statement in py39 to check for failure

* modify the if greq

* =~ is not supported in github

* add back if grep because the shell is bash

* revert format

* change runner os back to window and install py27

* refreshevn was giving error, so change it to use path directly

* add github cached for python27 installation

* revert changes to avoid cached, it gave out some errors
  • Loading branch information
feng-j678 authored Jun 21, 2023
1 parent ef693ec commit 565390c
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -50,20 +50,22 @@ jobs:
id: getoutput
run: echo "::set-output name=contents::$(cat err.txt)"
- name: Check if all tests passed
if: contains( steps.getoutput.outputs.contents , 'FAILED (failures=' )
if: contains( steps.getoutput.outputs.contents, 'FAILED (failures=' )
shell: cmd
run: |
echo "${{ steps.getoutput.outputs.contents }}"
exit 1
codecov-python-27:
runs-on: windows-latest
needs: codecov-python-39
steps:
- name: Checkout
uses: actions/checkout@v3
- name: Set up Python 2.7
uses: actions/setup-python@v4
with:
python-version: 2.7
run : |
choco install python2 -y
SETX PATH "%PATH%;C:\Python27"
- name: Install dependencies
run: pip install coverage
- name: Run extension tests
Expand Down

0 comments on commit 565390c

Please sign in to comment.