Skip to content

ci: disable ci#58

Open
emjshrx wants to merge 1 commit into
masterfrom
disable-ci
Open

ci: disable ci#58
emjshrx wants to merge 1 commit into
masterfrom
disable-ci

Conversation

@emjshrx
Copy link
Copy Markdown
Contributor

@emjshrx emjshrx commented Dec 10, 2025

disable all CI pipelines so that it can only be run manually. CI pipeline runs cost money and are proably billed to Bitshala-Incubator organisation

Copy link
Copy Markdown
Collaborator

@theanmolsharma theanmolsharma left a comment

Choose a reason for hiding this comment

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

Strong NACK. We are working on fixing the CI so that it can be useful.
I am not worried about billing since we haven't exceeded the free tier (yet). When we do, the org can cover the CI bill. GitHub Actions is relatively cheap.

@emjshrx
Copy link
Copy Markdown
Contributor Author

emjshrx commented Dec 11, 2025

Strong NACK. We are working on fixing the CI so that it can be useful. I am not worried about billing since we haven't exceeded the free tier (yet). When we do, the org can cover the CI bill. GitHub Actions is relatively cheap.

  1. This doesnt delete the CI, it just disables auto runs. Can be turned on once its fixed
  2. The only useful ci anyways is ‘ .github/workflows/ci.yml’ and the other pipelines are for deploying to the app stores and that shouldn’t be done on every pull request anyways

@theanmolsharma
Copy link
Copy Markdown
Collaborator

theanmolsharma commented Dec 11, 2025

The PRs for fixing the CIs are already up. Let's get back to this and once they are merged. Having said that, I agree with point 2. According to what @chaitika told me, the other pipelines only build and validate the build, they don't upload it to PlayStore. If that's not the case then we should disable them.

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