Skip to content

Simplify CI/CD: single job, main-only, no matrix#50

Merged
BEKO2210 merged 1 commit intomainfrom
claude/add-italian-french-languages-WV9NS
Mar 10, 2026
Merged

Simplify CI/CD: single job, main-only, no matrix#50
BEKO2210 merged 1 commit intomainfrom
claude/add-italian-french-languages-WV9NS

Conversation

@BEKO2210
Copy link
Owner

  • Remove pull_request trigger (only push to main + manual dispatch)
  • Remove Node version matrix (single Node 22.x)
  • Merge build-and-test + deploy into one job
  • Remove oversized icon deletion (file already removed from repo)
  • Remove npm audit step (non-blocking, better as separate workflow)

https://claude.ai/code/session_01Xgwdd2K7PAzPhq562NEtNJ

Description

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Performance improvement
  • CI/CD changes

Related Issues

Screenshots

Before After

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • All UI text is in German with correct umlauts (or uses i18n keys)
  • I have added translation keys for both de and en if new text was added
  • npm run build completes without errors
  • npm run test passes all tests
  • npx tsc --noEmit reports no type errors
  • No unused imports or variables (noUnusedLocals: true)
  • Tailwind classes used (no separate CSS files)
  • Lucide React icons used (no other icon libraries)

Testing

  • Tested in Chrome
  • Tested in Firefox/Safari
  • Tested on mobile
  • Tested offline functionality
  • Tested dark mode
  • Tested light mode

- Remove pull_request trigger (only push to main + manual dispatch)
- Remove Node version matrix (single Node 22.x)
- Merge build-and-test + deploy into one job
- Remove oversized icon deletion (file already removed from repo)
- Remove npm audit step (non-blocking, better as separate workflow)

https://claude.ai/code/session_01Xgwdd2K7PAzPhq562NEtNJ
@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@BEKO2210 BEKO2210 merged commit ddecca7 into main Mar 10, 2026
@BEKO2210 BEKO2210 deleted the claude/add-italian-french-languages-WV9NS branch March 10, 2026 15:42
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.

2 participants