Skip to content

Improves CI/CD and code quality with ESLint#62

Merged
GauteR merged 4 commits intomasterfrom
fix-ci
Aug 8, 2025
Merged

Improves CI/CD and code quality with ESLint#62
GauteR merged 4 commits intomasterfrom
fix-ci

Conversation

@GauteR
Copy link
Member

@GauteR GauteR commented Aug 8, 2025

This pull request:

  • Updates CI workflows to align with the master branch and removes the code-quality.yml workflow.
  • Implements a comprehensive ESLint configuration to enforce code style and best practices.
  • Addresses code quality issues identified during linting by applying formatting and style rule changes.

GauteR added 3 commits August 8, 2025 14:16
- Add a new ESLint configuration file to enforce coding standards and best practices.
- Update package.json to include @eslint/js and globals as development dependencies.
- Upgrade eslint to version 9.32.0 for improved linting capabilities.
- Modify various source and test files to comply with the new ESLint rules, including adjustments to eslint-disable comments for better clarity and consistency.
@socket-security
Copy link

socket-security bot commented Aug 8, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedglobals@​13.24.0 ⏵ 16.3.010010084 +189100
Updated@​eslint/​js@​8.57.0 ⏵ 9.32.010010090 +594100
Updatedeslint@​8.57.0 ⏵ 9.32.097 +110010094 +1100

View full report

@github-actions
Copy link

github-actions bot commented Aug 8, 2025

PR Check Results

✅ Passed Linting
✅ Passed Tests
📊 Coverage: 69.83%

✅ Coverage meets requirements

- Install pnpm in both build and runner stages for consistent package management.
- Update dependency installation to use pnpm with a frozen lockfile for production.
- Create a non-root user for improved security in the runner stage.
- Adjust health check command for better reliability with retries.
- Ensure proper ownership of copied files in the runner stage.
@github-actions
Copy link

github-actions bot commented Aug 8, 2025

PR Check Results

✅ Passed Linting
✅ Passed Tests
📊 Coverage: 69.83%

✅ Coverage meets requirements

@GauteR GauteR merged commit 7f28702 into master Aug 8, 2025
9 checks passed
@GauteR GauteR deleted the fix-ci branch August 8, 2025 12:30
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