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 CI and migrate to Github Actions #58

Merged
merged 15 commits into from
Oct 16, 2023

Conversation

Stormheg
Copy link
Member

@Stormheg Stormheg commented Oct 5, 2023

This PR migrates all our continues integration to Github Actions.

Some of these changes are in the spirit of the Python Package Maintenance Guidelines that Wagtail Nest packages should follow.

  • Drop support for Python 3.7
  • Update license to include that wagtail-ab-testing is now part of Wagtail Nest
  • Remove Travis CI config
  • Remove CircleCI config
  • Clean up tox config and fix some minor issues with running tests
  • Move nightly test against Wagtail's main branch to Github Actions
  • Move Javascript formatting checks to Github Actions
  • Move Python checks to Github Actions
  • Update readme badges
  • Make sure the readme renders on PyPI.org
  • Add section on where to report security issues

@Stormheg Stormheg force-pushed the chore/ci-update branch 14 times, most recently from c4b13b8 to d518b57 Compare October 6, 2023 12:18
Split out test against Wagtail main. This way we can call it seperately
and it won't fail our entire CI if it ever fails.
Django asks us if we want to re-create the test database and waits for
input.
Previously, tox environments were competing for access to the same
database. Now they all have their own database and don't conflict.
Split test against wagtaimain into a postgres and a sqlite variant.
@Stormheg Stormheg marked this pull request as ready for review October 6, 2023 12:21
@Stormheg
Copy link
Member Author

Thanks @SaptakS!

@Stormheg Stormheg merged commit 6aaccfe into wagtail-nest:main Oct 16, 2023
@Stormheg Stormheg deleted the chore/ci-update branch October 16, 2023 13:34
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