Skip to content

Comments

ci: add CI workflow for PRs and main branch#3

Closed
init4samwise wants to merge 1 commit intomasterfrom
samwise/eng-1913-enable-ci
Closed

ci: add CI workflow for PRs and main branch#3
init4samwise wants to merge 1 commit intomasterfrom
samwise/eng-1913-enable-ci

Conversation

@init4samwise
Copy link

Summary

Adds GitHub Actions CI workflow for the blockscout repo.

Changes

New .github/workflows/ci.yml with:

  • Compile — Compiles with --warnings-as-errors
  • Format — Checks code formatting with mix format --check-formatted
  • Credo — Runs linting with mix credo --strict
  • Test — Runs tests with PostgreSQL service container

Triggers

  • Push to master or main
  • PRs targeting master or main

Notes

Uses Elixir 1.19.4 and OTP 27.3.4.6 (matching existing config.yml).

Closes ENG-1913

Adds GitHub Actions CI workflow with:
- Compile (with warnings-as-errors)
- Format check
- Credo linting (strict mode)
- Tests with PostgreSQL service

Closes ENG-1913
@init4samwise
Copy link
Author

Closing this PR because CI is already enabled via the existing config.yml workflow inherited from upstream Blockscout.

The existing workflow (config.yml) already provides:

  • ✅ Compile
  • ✅ Format check (mix format --check-formatted)
  • ✅ Credo (linting)
  • ✅ Dialyzer (type checking)
  • ✅ Comprehensive tests (EthereumJSONRPC, Explorer, Indexer, BlockscoutWeb)
  • ✅ Additional checks: ESLint, Jest, Sobelow security analysis, spell checking

All these checks are passing on PRs to master. No additional CI workflow is needed.

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.

1 participant