-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Add filepath to extract-meta errors #3364
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
base: dev
Are you sure you want to change the base?
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 🫠 |
Fixes #3362