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 basic CI #51

Closed
wants to merge 1 commit into from
Closed

Conversation

jamesgeddes
Copy link

Add a bare-bones CircleCI configuration that,

  • runs test.py
  • checks that the cheese shop currently has an installable version.

CircleCI costs $0 for open-source projects and is pretty good. I notice that this project has no CI in place, and if it did I would not have spent so much time working out how to install it.

Could do with some fleshing out so I'm open to suggestions.

Add a bare-bones CircleCI configuration that,
- runs test.py
- checks that the cheese shop currently has an installable version.
@zdohnal
Copy link
Member

zdohnal commented Apr 6, 2023

Hi @jamesgeddes ,

thank you for the PR! It pushed me to finally activate the Github Actions, because that's the CI OpenPrinting group uses, so I would like to stick with that.

For that reason, I'm sorry for not merging the PR.

@zdohnal zdohnal closed this Apr 6, 2023
@jamesgeddes
Copy link
Author

Oh no need to apologise, I did not realise that there was already a standard. Always best to stick with the existing standards. Glad the PR was at least helpful in some way, so thanks for taking the time to consider it.

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