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

Add automated release system #67

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

dinigo
Copy link

@dinigo dinigo commented Sep 5, 2022

There are a couple of changes here:

In order to release more frequently it is necesary to automate some tasks. Linting and Test running are already. However deployment is not.

In order for contributors to have a common workflow/ci-cd platform, and provided that this is hosted in github, as well as 99% of FOSS projects. I propose migrating the ci to github workflows.

I moved both pre-exising workflows from circle-ci to github so we have them all in the same place

The purpose for this is to stablish a good basis to close several features contributed in #48 and also frequent user requests like #65

@dinigo
Copy link
Author

dinigo commented Sep 5, 2022

Here the build results https://github.com/teamdatatonic/airflow-dbt/actions/runs/2993755238 for a pick

Copy link
Contributor

@RafalSiwek RafalSiwek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will be helpful, LGTM!

@dinigo
Copy link
Author

dinigo commented Sep 6, 2022

@RafalSiwek , a couple of actions have to be taken from Gocardless side:

  • Add Pypi token to the repo so the job can authenticate
  • Switch from circle-ci to github workflows
  • Switch to the prod pypi: You can see that it is uploading the package to the TEST pypi repo for now. Once it's working do a small commit just remove this line and it will use the prod by default

@emily-flambe
Copy link

Do we have any ETA on when the next release will be ready?

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

3 participants