-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add sceneryStack to CT #226
Comments
I believe @jonathanolson and I want to first try to proceed by adding scenerystack repos to active-repos. |
|
The repos are linting fine, HOWEVER since many of them are stand-alone (by definition can't rely on existence of perennial/perennial-alias), they need to reference dependencies from package.json in their eslint configs, and therefore need to be @zepumph recently noted:
So therefore I've disabled the linting of those files in lint-main (noted by @zepumph above at https://github.com/phetsims/perennial/blob/67147519c7434b3c6f369d4d7035c60c8ab24077/js/eslint/lint-main.ts#L35). Should be good to discuss how to support this. |
Some test issues noted in #230. |
Taking a look here. |
|
Above @jonathanolson and I worked to add an ability to kill the child process when the timeout occurs during an npm run. I believe this will help fix the issues we have been having with CT unable to remove old snapshots. |
perennial has been failing CT on every cycle, due to this issue according to @zepumph. Details below.
|
No description provided.
The text was updated successfully, but these errors were encountered: