Repository navigation
fix: say which requirement has no license when packaging stops - #79
Merged
Merged
Conversation
packageToolbox adds each GitHub requirement to the toolbox as additional software with the URL of its license file. When GitHub finds no license for one of them, its /license endpoint returns 404 and packaging stopped with the raw webread error, which doesn't say what to do: The server returned the status 404 with message "Not Found" in response to the request to URL https://api.github.com/repos/VH-Lab/pyraview/license. getLicenseHtmlUrl now turns that 404 into a MatBox:GitHub:LicenseNotFound error naming the repository and asking for a LICENSE file. Packaging still stops, as it should: MATLAB needs a valid license URL for each additional software package, shown to the user during installation. Not run: MATLAB is not available where this was written. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01W156oKLRkXpxcrNEtFtyxH
ehennestad
enabled auto-merge (squash)
October 26, 2026 19:41
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
packageToolbox adds each GitHub requirement to the toolbox as additional software with the URL of its license file. When GitHub finds no license for one of them, its /license endpoint returns 404 and packaging stopped with the raw webread error, which doesn't say what to do:
The server returned the status 404 with message "Not Found" in
response to the request to URL
https://api.github.com/repos/VH-Lab/pyraview/license.
getLicenseHtmlUrl now turns that 404 into a MatBox:GitHub:LicenseNotFound error naming the repository and asking for a LICENSE file. Packaging still stops, as it should: MATLAB needs a valid license URL for each additional software package, shown to the user during installation.
Not run: MATLAB is not available where this was written.
Claude-Session: https://claude.ai/code/session_01W156oKLRkXpxcrNEtFtyxH