1
0
Fork 0
mirror of https://github.com/imjasonh/apidiff-action synced 2026-07-06 22:52:43 +00:00
Commit graph

15 commits

Author SHA1 Message Date
Jason Hall
b91371174f
refactor: simplify dist management to match GitHub Actions template
- Remove pre-commit hooks for building/checking dist
- Add check-dist.yml workflow that validates dist in CI
- This matches the approach used by actions/javascript-action
- Eliminates OS-specific build differences affecting commits
2025-07-30 09:53:16 -04:00
Jason Hall
113224a225
fix: ensure pre-commit hooks use Node 20
- Create dedicated scripts for pre-commit hooks that source nvm
- Ensure dist is always built with Node 20 to match CI environment
- This prevents Node version mismatches between local and CI builds
2025-07-30 09:32:45 -04:00
Jason Hall
051f997977
Enforce Node 20 in pre-commit hooks
- Add Node version check to build-dist pre-commit hook
- Create helper script to switch to Node 20
- Update documentation with clear instructions
- Fix YAML syntax in pre-commit config

This ensures dist is always built with Node 20, preventing CI failures
due to version mismatches.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-07-30 09:22:44 -04:00
Jason Hall
9f05344e16
Configure Node.js v20 for local development
- Add .node-version and .nvmrc files specifying Node 20.9.0
- Add engines field to package.json requiring Node 20
- Re-enable dist build/check hooks in pre-commit
- Add build-dist workflow for automated builds with Node 20
- Update documentation with Node version requirements

This ensures consistent builds between local development and CI.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-07-30 09:13:03 -04:00
Jason Hall
1cabe34a14
Standardize on Node.js v20 and disable dist checks
- Keep action.yml using node20 (GitHub Actions supported version)
- Document requirement to use Node v20 for development
- Disable pre-commit dist build/check hooks since output varies between Node versions
- Update CLAUDE.md with Node version requirements

This ensures consistency between local development and CI.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-07-30 09:02:24 -04:00
Jason Hall
4fb67b86d5
Re-enable check-dist hook now that dist/ is committed 2025-07-30 08:56:03 -04:00
Jason Hall
25b3f0a37e
Prepare to commit dist/index.js
- Remove dist/ from .gitignore
- Exclude dist/ from pre-commit hooks
- Exclude *.md from prettier formatting
- Update CLAUDE.md with build documentation
- Temporarily disable check-dist hook

This prepares for committing the compiled dist/index.js file.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-07-30 08:55:35 -04:00
Jason Hall
796d4dde5c
Exclude package-lock.json from pretty-format-json hook
This prevents Dependabot PRs from failing CI due to package-lock.json
formatting differences. The file is auto-generated by npm and its exact
formatting is not important as long as it's valid JSON.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-07-30 08:29:39 -04:00
Jason Hall
f9bedf8b44
Use pre-commit action in CI
- Replace individual lint/test/build steps with pre-commit action
- Ensures CI runs exactly the same checks as local development
- Includes dist up-to-date check via pre-commit hooks
2025-07-29 17:21:43 -04:00
Jason Hall
805c061497
Fix lint errors and formatting
- Remove unused variables
- Add .eslintignore to exclude dist/
- Apply prettier formatting
- Fix all pre-commit hook issues
2025-07-29 17:07:15 -04:00
Jason Hall
d05e5461cf
Merge main and resolve conflicts
Keep Node.js-based configuration for both CI and pre-commit
2025-07-29 16:44:48 -04:00
Jason Hall
b92e3a611c
Transform to apidiff-action GitHub Action
- Remove all Go code and replace with Node.js GitHub Action
- Use official golang.org/x/exp/cmd/apidiff tool
- Add PR comment functionality
- Support both git refs and directory comparisons
- Add comprehensive test suite with testdata examples
- Update CI to test action with breaking/safe/clean examples

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-07-29 16:33:49 -04:00
Jason Hall
24ccbeb042
Remove golangci config to support v2
golangci-lint v2.3.0 works best with default configuration

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-07-29 15:01:40 -04:00
Jason Hall
986abe49b3
Fix CI: Install goimports and add golangci-lint to pre-commit
- Add goimports installation to CI workflow
- Add golangci-lint to pre-commit hooks
- Fix deprecated golangci-lint config options (gomnd→mnd, check-shadowing→shadow)
- Remove duplicate issues section in golangci.yml

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-07-29 14:54:53 -04:00
Jason Hall
37ea620324
Add CI/CD setup with pre-commit hooks and GitHub Actions
- Add pre-commit configuration for code quality checks
- Set up GitHub Actions for CI with test, lint, and build jobs
- Add golangci-lint configuration
- Create Makefile for common development tasks
- Update README with development instructions
- Fix code formatting issues

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-07-29 14:38:27 -04:00