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

Move to GitHub Actions #521

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Conversation

robertodr
Copy link
Collaborator

Let's fix CI, which is currently rotten and useless for incoming PRs.

Description

  • Use Poetry to manage Python requirements
  • Move to GitHub actions for testing
    • Linux with GNU toolchain
    • Linux with Intel toolchain
    • Linux with PGI toolchain
    • macOS
    • Windows

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Status

  • Ready to go

@robertodr robertodr marked this pull request as draft June 18, 2020 17:02
@bast
Copy link
Member

bast commented Jun 23, 2020

Thanks for starting this. I will soon help out here. I agree that we need to restore testing otherwise it's like flying through the fog without instruments.

@robertodr
Copy link
Collaborator Author

Not much luck, currently, but that's because testing has many layers for this project.

First strep is to green up Linux/macOS and phase out Travis. Then phase out CircleCI (which is running custom PGI and Intel images, both rotten...) Phase out of AppVeyor comes last and I believe that will be the greatest amount of work...

What I would like to achieve:

  1. Unpack testing of each chapter in its own sub-action. It's easier to see what's failing and each chapter gets a badge.
  2. Redo Docker images for use with Actions, so that we can also "unpack" successes/failures by Docker image.

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.

None yet

2 participants