Conversation
|
@T4rk1n , is it possible to generate a tarball with a prerelease with these changes? Would facilitate testing early. |
|
This PR looks great to me! I actually ran into an error like the one reported in issue 3362 when I was working on DMC recently. It failed during the build with this message and I was stuck.
To test this PR, I checked out that failing commit in DMC and tried it with this solution. There was no error message like the one above during the build, so I could take the next step and run the tests. I was able to see this helpful error message. This PR would have saved me lots of time! @ndrezn do you need any other scenarios tested? |
|
I don't recall actually why I wanted early testing on this 🫠 |
|
There is extra changes in this PR that needs removed because I was trying to fix the CI. While it skips: It should emit a warning but no error, think we can get the parent and the previous prop name to help identify the invalid prop. |
Fixes #3362