-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Update eslint plugins, heed its warnings; mostly about structuredCloned (TIL) #6483
base: main
Are you sure you want to change the base?
Conversation
Well, Cypress is unhappy with:
|
so we probably have been json to/fromming all manner of junk that doesn't rounttrip through json since forever |
So in a couple of these a function gets stored in This is basically a way to get special property settings specific to this context without having to store things in But of course this causes issues when using The function getting lost during copy isn't a problem, because at that point the compilers have already been instantiated including buildenvsetup. That's why We can probably just use the function directly instead of storing it. Gonna give that a try. |
Those are fixed and simplified. But the Tools remain, they are classes (they get replaced again later) Bit too complicated to fix that at the moment. Basically I'm still reverting the change in options-handler. |
Also contains:
String.raw
(TIL) - in since node.js 4.x...result
fixes in arg parsers, bug perpetrated by yours truly.