Skip to content

Conversation

@i1skn
Copy link
Contributor

@i1skn i1skn commented Feb 2, 2026

Fixes the lint:js:ci script

What changed (plus any additional context for devs)

  • Removed src/graphql/index.ts from the list of allowed barrel files in .eslintrc.js and added an eslint-disable comment and TODO note in src/graphql/index.ts as the package has it's own eslint config and file resolution is relative to it
  • Fixed the lint:js:ci script in package.json by adding the missing . path parameter

What to test

  • Verify that linting still works correctly with yarn lint
  • Ensure CI linting passes with the updated lint:js:ci command

Copy link
Contributor Author

i1skn commented Feb 2, 2026

@i1skn i1skn marked this pull request as ready for review February 2, 2026 18:39
@i1skn i1skn force-pushed the 02-02-chore_fix_lint_on_ci branch from d840b5f to 4449aa3 Compare February 2, 2026 18:58
@github-actions
Copy link

github-actions bot commented Feb 2, 2026

🧪 Flashlight Performance Report (AWS Device Farm)

🔀 Commit: 0fe8bb9

📎 View Artifacts

Metric Current Δ vs Baseline
Time to Interactive (TTI) 5555 ms 🔴 +59.8 ms (+1.1%)
Average FPS 56.74 🟢 +1.2 (+2.1%)
Average RAM 390.4 MB ⚪ -1.7 MB (-0.4%)

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Launch in simulator or device for 0fe8bb9

@i1skn i1skn merged commit 5433833 into develop Feb 3, 2026
21 of 22 checks passed
@i1skn i1skn deleted the 02-02-chore_fix_lint_on_ci branch February 3, 2026 15:11
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.

3 participants