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

Initial circle setup #12

Merged
merged 9 commits into from
Feb 10, 2022
Merged

Initial circle setup #12

merged 9 commits into from
Feb 10, 2022

Conversation

ArayB
Copy link
Contributor

@ArayB ArayB commented Feb 10, 2022

Adds Circle CI to the repo.

Currently just runs the tests. The repo needs to have rubocop run over it and then we can enable that in CI too.
I didn't do it in this PR as there would be lots of rubocop change obscuring the CI setup changes.

Environment variables have been added for database users and passwords. These will need to be added to you local .env file after merging this PR.

@ArayB ArayB self-assigned this Feb 10, 2022
@ArayB ArayB marked this pull request as draft February 10, 2022 13:11
@ArayB ArayB marked this pull request as ready for review February 10, 2022 13:13
@raspberrypigithubbot
Copy link

  • CircleCI build #9
  • Test coverage: 62.33%

@raspberrypigithubbot
Copy link

Copy link
Contributor

@loiswells97 loiswells97 left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

@ArayB ArayB merged commit eb160c0 into main Feb 10, 2022
@ArayB ArayB deleted the setup-circle branch February 10, 2022 15:21
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.

3 participants