Skip to content

Enable integration tests#40

Merged
Ethan-Arrowood merged 15 commits intomainfrom
fix/ci-integration-tests
May 1, 2026
Merged

Enable integration tests#40
Ethan-Arrowood merged 15 commits intomainfrom
fix/ci-integration-tests

Conversation

@Ethan-Arrowood
Copy link
Copy Markdown
Member

@Ethan-Arrowood Ethan-Arrowood commented Apr 30, 2026

Integration tests on CI are fixed.

Solidifies the repo too with nvmrc, devengines, and package-lock

Removes prettier since it was causing installation issues that were very annoying

Can remove the || true in the build step once we fix harper exports

@socket-security
Copy link
Copy Markdown

socket-security Bot commented Apr 30, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedharper@​5.0.3 ⏵ 5.0.77310091 +196 +1100
Updated@​harperfast/​integration-testing@​0.2.0 ⏵ 0.3.078 +310010094 +5100
Updatedtypescript@​5.9.3 ⏵ 6.0.3100 +110090 +19990
Updated@​playwright/​test@​1.52.0 ⏵ 1.59.110010010099100

View full report

@socket-security
Copy link
Copy Markdown

socket-security Bot commented May 1, 2026

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn High
License policy violation: npm typescript

License: LicenseRef-W3C-Community-Final-Specification-Agreement - The applicable license policy does not permit this license (5) (package/ThirdPartyNoticeText.txt)

From: package-lock.jsonnpm/typescript@6.0.3

ℹ Read more on: This package | This alert | What is a license policy violation?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Find a package that does not violate your license policy or adjust your policy to allow this package's license.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/typescript@6.0.3. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

@Ethan-Arrowood Ethan-Arrowood marked this pull request as ready for review May 1, 2026 02:32
@Ethan-Arrowood Ethan-Arrowood changed the title Enable integration tests (as-is) Enable integration tests May 1, 2026
@Ethan-Arrowood Ethan-Arrowood merged commit a52c3c7 into main May 1, 2026
6 checks passed
@Ethan-Arrowood Ethan-Arrowood deleted the fix/ci-integration-tests branch May 1, 2026 13:02
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