Skip to content

Fix CI + Tidelift + lockfile [2.x.x]#28

Open
MoOx wants to merge 4 commits into
2.x.xfrom
2.x.x-fix
Open

Fix CI + Tidelift + lockfile [2.x.x]#28
MoOx wants to merge 4 commits into
2.x.xfrom
2.x.x-fix

Conversation

@MoOx
Copy link
Copy Markdown
Collaborator

@MoOx MoOx commented May 28, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 28, 2026 06:39
Copy link
Copy Markdown

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 modernizes the project’s maintenance setup by moving CI off Travis and onto GitHub Actions, adding Tidelift security reporting guidance, and committing an npm lockfile to support repeatable installs.

Changes:

  • Replace Travis CI with a GitHub Actions workflow for npm test on Ubuntu and Windows.
  • Update README badges and add a Tidelift security contact section.
  • Add .gitattributes and commit a new package-lock.json.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
README.md Refreshes project badges and adds security vulnerability reporting instructions via Tidelift.
package-lock.json Introduces an npm lockfile intended to make installs deterministic.
.travis.yml Removes the legacy Travis configuration.
.github/workflows/test.yml Adds GitHub Actions CI workflow to run tests on push/PR.
.gitattributes Enforces LF line endings across the repository.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/test.yml Outdated
MoOx and others added 2 commits May 28, 2026 10:09
The `BABEL_ENV=publish` inline syntax fails on Windows cmd.exe.
Wrapping with cross-env makes the prepublish script cross-platform.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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