Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating to use GithubActions, latest Tap, and complete coverage #11

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DavidTanner
Copy link

I'm hoping this library can be updated to resolve Snyk issues with Glob, inflight and others.

@DavidTanner
Copy link
Author

@wraithgar, or @isaacs is this project intended to go on, or should it be removed from other Open source projects?

@isaacs
Copy link
Owner

isaacs commented Dec 27, 2023

What is the update here? Why is the code changing without any functionality change?

@DavidTanner
Copy link
Author

This PR removes the travis config, and replaces it with GithubActions. Which will need to be enabled.
It also updates tap from v2 to v18, which then required full coverage in the tests.

Follow on updates would be to convert to ESM.

I am trying to address a SNYK issue regarding inflight, https://security.snyk.io/vuln/SNYK-JS-INFLIGHT-6095116, and followed the dependencies to here and once.

I don't know what protocol you want followed here, and I am open to pretty much anything.

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.

None yet

2 participants