Skip to content
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

Merged
merged 30 commits into from
Dec 17, 2024

Conversation

johnsimons
Copy link
Member

@johnsimons johnsimons commented Dec 2, 2024

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:

image

@johnsimons johnsimons self-assigned this Dec 2, 2024
@johnsimons johnsimons marked this pull request as ready for review December 2, 2024 23:34
@johnsimons
Copy link
Member Author

@PhilBastian I am not 100% sure whether the title of the page should still be "Thrid-party licenses", thoughts?

@johnsimons
Copy link
Member Author

I will update the licenses text to the new URL once this is merged

@irinascurtu
Copy link
Contributor

@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?

Copy link
Member

@udidahan udidahan left a 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?

platform/third-party-licenses.md Outdated Show resolved Hide resolved
platform/third-party-licenses.md Outdated Show resolved Hide resolved
@johnsimons
Copy link
Member Author

@udidahan

Also, just to verify, have links to the deleted files been updated appropriately?

I have added a redirect, and will also update all locations found in this search once this is live.

@udidahan
Copy link
Member

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.

@johnsimons johnsimons merged commit 450d6f5 into master Dec 17, 2024
4 checks passed
@johnsimons johnsimons deleted the john/thrid branch December 17, 2024 00:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants