Skip to content

Update CI/CD workflows, dependencies, and add tests - #154

Merged
saumier merged 12 commits into
reconciliation-api:masterfrom
culturecreates:master
Jul 15, 2026
Merged

Update CI/CD workflows, dependencies, and add tests#154
saumier merged 12 commits into
reconciliation-api:masterfrom
culturecreates:master

Conversation

@AbhishekPAnil

Copy link
Copy Markdown
Contributor

This pull request introduces several improvements to code quality, testing, and CI/CD automation for the project. The most notable changes include adding ESLint configuration for consistent code style, enhancing test coverage for key components, and updating CI workflows to better support Dependabot and Node.js caching. Additionally, minor code cleanups and accessibility improvements are made throughout the codebase.

Code Quality and Testing Enhancements:

  • Added a new .eslintrc.cjs file to enable ESLint with recommended rules for React, React Hooks, and accessibility, and disabled unused PropTypes rule for this legacy codebase.
  • Improved and expanded tests for App and ReconciliationServiceInput components, including mocking network requests and simulating user input. [1] [2]

CI/CD and Automation Improvements:

  • Added a workflow .github/workflows/dependabot-auto-merge.yml to automatically merge Dependabot PRs for patch and eligible minor development updates.
  • Updated .github/workflows/ci.yml to add minimal permissions, enable npm caching in actions/setup-node, and remove the npm test step (possibly to be replaced by other testing workflows). [1] [2]
  • Improved .github/dependabot.yml to group patch and minor updates separately, limit open PRs, and ignore major version updates for less noise.

Codebase Maintenance and Minor Fixes:

  • Upgraded vite dependency from ^7.3.3 to ^7.3.6 in package.json.
  • Fixed typos, improved accessibility (e.g., aria-label in FeatureCell), and made minor code style corrections (e.g., consistent use of single quotes, arrow function simplification, and HTML entity corrections). [1] [2] [3] [4] [5] [6] [7] [8] [9]

These changes collectively improve the maintainability, reliability, and developer experience for the project.

AbhishekPAnil and others added 12 commits June 16, 2026 14:47
Bumps [undici](https://github.com/nodejs/undici) from 7.24.1 to 7.28.0.
- [Release notes](https://github.com/nodejs/undici/releases)
- [Commits](nodejs/undici@v7.24.1...v7.28.0)

---
updated-dependencies:
- dependency-name: undici
  dependency-version: 7.28.0
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Add initial tests and clean up code
Bumps the patch-updates group with 1 update: [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite).


Updates `vite` from 7.3.3 to 7.3.6
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/v7.3.6/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v7.3.6/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-version: 7.3.6
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: patch-updates
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…dici-7.28.0

chore(deps): bump undici from 7.24.1 to 7.28.0
@saumier
saumier merged commit 14868d1 into reconciliation-api:master Jul 15, 2026
2 checks passed
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