gitcoinco / gitcoinco/grants-stack-indexer

Add PR checks that don't depend on fly.io

Open
#145 1 comment 0 reactions 0 assignees View on GitHub
ci good first issue
Dominant language
TypeScript
Stars
35
Forks
27
PR merge metrics
No merged PRs in 30d

Description

Task: Currently we use `flyctl deploy --remote-only --build-only` in `.github/workflows/ci.yml` both to build and to run checks, however that presupposes the presence of a FLY_API_TOKEN, which might not be there for contributed PRs.

Changes:
- Add an explicit lint + test step in the workflow that can be always run.
- Invoke `flyctl` for actual deployment only.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.