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

IG: Delete circleci as we are well covered without it #6955

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

mahf708
Copy link
Contributor

@mahf708 mahf708 commented Jan 29, 2025

Delete circleci as we are well covered without it. This gets rid of circleci testing, by deleting its folder. We are currently abundantly covered for what circleci testing offers us (via github actions; in fact, for github actions, not only do we build, we run several tests for free).

[BFB]

@mahf708 mahf708 added BFB PR leaves answers BFB Testing Anything related to unit/system tests labels Jan 29, 2025
@rljacob
Copy link
Member

rljacob commented Jan 29, 2025

I turned it off at CircleCI too.

rljacob added a commit that referenced this pull request Jan 30, 2025
Delete circleci as we are well covered without it. This gets rid of circleci testing, by deleting its folder.
We are currently abundantly covered for what circleci testing offers us (via github actions;
in fact, for github actions, not only do we build, we run several tests for free).

[BFB]
@rljacob rljacob merged commit 1c6eb4d into master Jan 30, 2025
2 checks passed
@rljacob rljacob deleted the mahf708/ig/buy-buy-circle-ci branch January 30, 2025 05:02
@mahf708
Copy link
Contributor Author

mahf708 commented Jan 30, 2025

Thanks @rljacob ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BFB PR leaves answers BFB Testing Anything related to unit/system tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants