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

test: make package json command check min files #2130

Closed
wants to merge 1 commit into from

Conversation

Techn1x
Copy link
Contributor

@Techn1x Techn1x commented Jan 19, 2024

Summary

See #1936

Description

I am hopeful that this can be used as a failing test (circle CI should run npm run build which will run these check commands and fail)

Screenshots (if appropriate):

Screenshot 2024-01-19 at 8 34 12 pm

@Techn1x
Copy link
Contributor Author

Techn1x commented Feb 5, 2024

@marcolink is this useful or should I close it?

@marcolink
Copy link
Member

cc @mgoudy91

@Techn1x
Copy link
Contributor Author

Techn1x commented Feb 19, 2024

Would be great to see this fixed for a patch version. I did give it a go, but was only able to come up with the failing test.

@axe312ger
Copy link
Collaborator

axe312ger commented Apr 25, 2024

@axe312ger axe312ger closed this Apr 25, 2024
@Techn1x Techn1x deleted the add-failing-test branch May 3, 2024 03:32
@Techn1x
Copy link
Contributor Author

Techn1x commented May 3, 2024

Thankyou! Can confirm underlying issue resolved in 10.9.0+

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.

3 participants