Skip to content

Conversation

yvalentin
Copy link
Contributor

No description provided.

@yvalentin yvalentin requested review from ttdm and Copilot September 15, 2025 07:50
@yvalentin yvalentin self-assigned this Sep 15, 2025
Copy link
Contributor

@Copilot 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

This PR enforces a consistent curly brace style across the codebase by adding curly braces to all single-statement control structures and configuring ESLint to enforce this pattern going forward.

  • Adds curly braces around all single-statement if/for/while blocks
  • Updates ESLint configuration with the "curly": ["error", "all"] rule
  • Maintains consistent code style and improves readability

Reviewed Changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.eslintrc.json Adds ESLint curly brace enforcement rule
libs/data/tests/data.test.ts Adds braces to if statement in error handling function
libs/data/src/program/yamlGenerator/eligibilityGenerator.ts Adds braces to early return condition
libs/data/src/program/mailManager.ts Adds braces to continue statement condition
libs/data/src/program/dataPipeline.ts Adds braces to logging conditions
libs/backend-ddd/src/opportunity/infrastructure/api/brevo/brevoDeal.ts Adds braces to error handling condition
libs/backend-ddd/src/common/domain/error/errors.ts Adds braces to early return in error function
apps/nuxt/src/tools/companyData/companyData.ts Adds braces to null check condition
apps/nuxt/src/tools/analytic/useExternalLinkTracker.ts Adds braces to early return condition
apps/nuxt/src/pages/TeeBudgetPage.vue Adds braces to canvas validation checks
apps/nuxt/src/components/project/details/ProjectTestimonies.vue Adds braces to computed property conditions
apps/nuxt/src/components/project/details/ProjectSideNav.vue Adds braces to computed property condition
apps/nuxt/src/components/program/detail/ProgramDetail.vue Adds braces to null check condition

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@yvalentin yvalentin removed the request for review from ttdm September 15, 2025 09:29
@yvalentin yvalentin merged commit 3dad4d2 into main Sep 15, 2025
12 checks passed
@yvalentin yvalentin deleted the feat/eslint-curly branch September 15, 2025 09:30
@yvalentin yvalentin linked an issue Sep 15, 2025 that may be closed by this pull request
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.

eslint - add curly rules
1 participant