Skip to content

Conversation

@AdekunleBamz
Copy link

This PR modernizes the project infrastructure and fixes testing ergonomics:

  • CI Integration: Added a Node.js CI workflow (.github/workflows/ci.yml) to ensure tests pass on modern Node versions (v20.x).
  • Test Script Fix: Removed the hard dependency on yarn in the npm test script. It now runs ava directly, allowing the tests to run smoothly regardless of the package manager used.
  • Documentation: Added a Build Status badge to the README.md.
  • Gitignore: Added .vscode/ to .gitignore to prevent editor configs from being committed.

@tonyofbyteball
Copy link
Member

Thanks for the PR.
We already have circleci. What's the use of the additional github actions ci?
The rest seems ok.

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