Skip to content

Fix GitHub Actions workflows#326

Merged
mdickinson merged 9 commits into
mainfrom
fix-ci
Feb 8, 2023
Merged

Fix GitHub Actions workflows#326
mdickinson merged 9 commits into
mainfrom
fix-ci

Conversation

@mdickinson
Copy link
Copy Markdown
Member

@mdickinson mdickinson commented Feb 7, 2023

This PR brings our continuous integration workflows back to working state

  • Don't try to run non-EDM-based tests on Python 3.6
  • Remove uses of GabrielBB/xvfb-action (we don't actually need a virtual framebuffer at all)
  • Temporarily pin the copyright end year; we'll fix the copyright headers in a followup PR
  • Update EDM and setup-edm-action versions
  • Other minor cleanups - e.g., don't update pip unnecessarily (the setup-python action already does this for us)

Checklist

  • Add a news fragment if this PR is news-worthy for end users. (see docs/releases/README.rst)

@mdickinson mdickinson marked this pull request as ready for review February 7, 2023 20:00
@mdickinson mdickinson changed the title WIP: Fix CI Fix GitHub Actions workflows Feb 7, 2023
@mdickinson mdickinson merged commit 7c86f16 into main Feb 8, 2023
@mdickinson mdickinson deleted the fix-ci branch February 8, 2023 08:22
corranwebster pushed a commit that referenced this pull request May 25, 2023
This PR brings our continuous integration workflows back to working
state

- Don't try to run non-EDM-based tests on Python 3.6
- Remove uses of GabrielBB/xvfb-action (we don't actually need a virtual
framebuffer at all)
- Temporarily pin the copyright end year; we'll fix the copyright
headers in a followup PR
- Update EDM and setup-edm-action versions
- Other minor cleanups - e.g., don't update `pip` unnecessarily (the
setup-python action already does this for us)

**Checklist**
- [ ] Add a news fragment if this PR is news-worthy for end users. (see
docs/releases/README.rst)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants