TST: Use Actions, update test matrix #38
Conversation
.github/workflows/ci_workflows.yml
Outdated
| matrix: | ||
| include: | ||
| - name: Python 3.6 and pytest 4.6 (Linux) | ||
| os: ubuntu-latest |
There was a problem hiding this comment.
name is not used. On doctestplus / remotedata I did not put a name because the os/python_version/toxenv combination seems clear enough. If there is a preference for a readable name, that's fine by me, but it should then probably be used to name the build (I guess name: ${{ matrix.name }} near runs-on) ?
There was a problem hiding this comment.
Name is used for display on the left navbar. But if you feel strongly about them, I can remove. I copy pasted from the Travis setup.
There was a problem hiding this comment.
Both are fine, it's just that it would be better to be consistent between repos.
For me the name does not add much information.
There was a problem hiding this comment.
For the sake of moving this forward, I removed the names. See https://github.com/pllim/pytest-openfiles/actions/runs/360370589
As advertised.
See https://github.com/pllim/pytest-openfiles/runs/1392224055?check_suite_focus=true