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

Update test infrastructure #250

Merged
merged 1 commit into from
Feb 15, 2023

Conversation

sirosen
Copy link
Collaborator

@sirosen sirosen commented Feb 6, 2023

  • update pre-commit config
  • switch from azure pipelines to GitHub Actions
  • other minor changes (e.g. tox.ini updates)

EDIT: Two parts of this struck as having been pulled into separate PRs. This is now just the move to GitHub Actions.

@sirosen
Copy link
Collaborator Author

sirosen commented Feb 6, 2023

@sloria, when you get a free minute, could you give me the permissions to update repo config?
I'd like to convert over to pre-commit.ci + github actions, to match what we did in webargs a while back.

It won't work with the current github config (which is looking for the Azure Pipelines check to pass), so I'd like to be able to get in there to update things.

I'm happy to work with what's here for now, but I want to be able to make the change.

This is a full conversion with nearly the same suite of builds,
updated to match the current tox config.

Some parts of the build are copied from webargs (where we initially
rolled out similar config) and are commented out for now.
@sirosen sirosen force-pushed the update-test-infrastructure branch from e6845ac to 320e9af Compare February 6, 2023 04:10
@sloria
Copy link
Member

sloria commented Feb 6, 2023

@sirosen sure thing; you should be good to go now

@sirosen sirosen merged commit 5e62889 into marshmallow-code:dev Feb 15, 2023
@sirosen sirosen deleted the update-test-infrastructure branch February 15, 2023 23:45
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