-
Notifications
You must be signed in to change notification settings - Fork 301
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
Updating third party docs page to include npm packages and products #6920
Conversation
the Setup project always fails
9a52ddf
to
92e3a84
Compare
3d17219
to
299c396
Compare
256bb64
to
3e4cd77
Compare
@PhilBastian I am not 100% sure whether the title of the page should still be "Thrid-party licenses", thoughts? |
I will update the licenses text to the new URL once this is merged |
@udidahan we need some advice here. Should we mention all the 3rd party libraries as a "Credit to these 3rd party" ? Has this a legal implication? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd use more neutral language here.
Also, just to verify, have links to the deleted files been updated appropriately?
Co-authored-by: Udi Dahan <[email protected]>
Co-authored-by: Udi Dahan <[email protected]>
I have added a redirect, and will also update all locations found in this search once this is live. |
OK, so while I can't say that I've reviewed all the other elements on this PR, I do approve the current state of the third-party-licenses.md file. |
This PR updates all the scripts to generate the 3rd party list to also include ServiceControl and ServicePulse.
The 3rd party list moves from
/nservicebus/upgrades/third-party-licenses
to/platform/third-party-licenses
.As part of this change we also improved the diff viewing when the changes are large, now we create a
changes.patch
artifact that is easier to review and apply manually.This also changes the layout of the page, the page changes to a simple list of all 3rd party libs that are used for all Particular software: