Skip to content

Run lint only on PRs, not on main#1

Merged
dfalling merged 2 commits into
mainfrom
ci-lint-pr-only
May 23, 2026
Merged

Run lint only on PRs, not on main#1
dfalling merged 2 commits into
mainfrom
ci-lint-pr-only

Conversation

@dfalling
Copy link
Copy Markdown
Owner

Summary

  • Gate the Biome lint step in ci.yml to pull_request events so it doesn't run on pushes to main
  • TypeScript checks, tests, and the Android build still run on both PRs and main

Test plan

  • On this PR: confirm the Biome (lint + format check) step runs
  • After merge: confirm the step is skipped on the main-branch push run

dfalling and others added 2 commits May 23, 2026 11:20
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Native module isn't registered in the Jest environment, so importing
EncryptedStorage threw "RNEncryptedStorage is undefined" and broke the
App test suite.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@dfalling dfalling merged commit 2bddc32 into main May 23, 2026
2 checks passed
@dfalling dfalling deleted the ci-lint-pr-only branch May 23, 2026 10:26
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