Skip to content

fix(ci): optomize ci with cache#1504

Open
NeonGamerBot-QK wants to merge 12 commits intomainfrom
ci-faster
Open

fix(ci): optomize ci with cache#1504
NeonGamerBot-QK wants to merge 12 commits intomainfrom
ci-faster

Conversation

@NeonGamerBot-QK
Copy link
Member

@NeonGamerBot-QK NeonGamerBot-QK commented Feb 24, 2026

This pr does:

  • makes db checks run only when db files are modified
  • makes scan_js use bun to be faster
  • uses cache for rails

@NeonGamerBot-QK NeonGamerBot-QK self-assigned this Feb 24, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the GitHub Actions CI workflow to improve caching/step consistency and streamline CI runs.

Changes:

  • Adds global Bundler environment configuration (path/clean/without) to support more consistent Bundler caching.
  • Simplifies several job steps (removes some step names, consolidates prettier checks, adds --prefer-offline for Yarn).
  • Reworks db_checks steps and reduces scan_js to a no-op.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@NeonGamerBot-QK NeonGamerBot-QK marked this pull request as ready for review February 24, 2026 03:52
@NeonGamerBot-QK NeonGamerBot-QK added this pull request to the merge queue Feb 24, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 24, 2026
@NeonGamerBot-QK NeonGamerBot-QK added this pull request to the merge queue Feb 24, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 24, 2026
@NeonGamerBot-QK NeonGamerBot-QK added this pull request to the merge queue Feb 25, 2026
@NeonGamerBot-QK NeonGamerBot-QK removed this pull request from the merge queue due to a manual request Feb 25, 2026
@cskartikey
Copy link
Member

Could you explain what changes were made and why they're necessary? Also, is this a standard or documented practice that I can review further?

ruby-version: .ruby-version
bundler-cache: true

- name: Scan for common Rails security vulnerabilities using static analysis
Copy link
Member

Choose a reason for hiding this comment

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

why are we removing the names?

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