Sync changes from GitHub Docs #1079
Merged
Conversation
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
There was a problem hiding this comment.
Pull Request Overview
This PR synchronizes changes from GitHub Docs, updating privacy policy documentation and legal terms. The changes include updates to subprocessor lists, cookie documentation improvements, and revisions to pre-release license terms.
- Updates the GitHub Subprocessors list with new providers and removes some entries
- Improves cookie documentation with detailed table of all cookies used by GitHub and third parties
- Modifies pre-release license terms to include defense obligations and AI service requirements
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| Policies/privacy-policies/github-subprocessors.md | Updates subprocessor list with new cloud infrastructure providers and removes some entries |
| Policies/privacy-policies/github-general-privacy-statement.md | Updates link reference to point to internal cookie documentation |
| Policies/privacy-policies/github-cookies.md | Complete rewrite with comprehensive cookie table and detailed descriptions |
| Policies/github-terms/github-pre-release-license-terms.md | Updates section 10 from "No Indemnity" to "Defense of Third Party Claims" with new requirements |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| | Microsoft (Azure) | Cloud Hosted Infrastructure, Data Hosting, AI Inference and AI Services | United States | United States | | ||
| | Microsoft (Azure) | Cognitive Services and customer support ticketing analysis | United States | United States | | ||
| | Microsoft | Technical Services | Australia, Brazil, Canada, France, Japan, Norway, Spain, Sweden, Switzerland, United Kingdom, United States | United States | | ||
| | Microsoft | Technical Services | Australia, Brazil, Canada, France, Japan, Norway, Spain, Sweden, Switzerland, United Kingdom, United States | United States| |
There was a problem hiding this comment.
Missing space before the closing pipe character. Should be "United States |" to maintain consistent table formatting.
Suggested change
| | Microsoft | Technical Services | Australia, Brazil, Canada, France, Japan, Norway, Spain, Sweden, Switzerland, United Kingdom, United States | United States| | |
| | Microsoft | Technical Services | Australia, Brazil, Canada, France, Japan, Norway, Spain, Sweden, Switzerland, United Kingdom, United States | United States | |
| If you have questions or concerns about a new subprocessor, please contact us via {% data variables.contact.contact_privacy %}. | ||
| GitHub provides a great deal of transparency regarding how we use your data, how we collect your data, and with whom we share your data. To that end, we provide this page which details how we use cookies. | ||
|
|
||
| GitHub uses cookies to provide and secure our websites, as well as to analyze the usage of our websites, in order to offer you a great user experience. Please take a look at our [Privacy Statement](/github/site-policy/github-privacy-statement#our-use-of-cookies-and-tracking-technologies) if you’d like more information about cookies, and on how and why we use them and cookie-related personal data. You can change your preference about non-essential cookies at any time by following [these instructions](/account-and-profile/setting-up-and-managing-your-personal-account-on-github/managing-personal-account-settings/managing-your-cookie-preferences-for-githubs-enterprise-marketing-pages). |
There was a problem hiding this comment.
The link to the Privacy Statement uses an inconsistent path format. It should use the same format as other links in the file ("/site-policy/privacy-policies/github-privacy-statement" instead of "/github/site-policy/github-privacy-statement").
Suggested change
| GitHub uses cookies to provide and secure our websites, as well as to analyze the usage of our websites, in order to offer you a great user experience. Please take a look at our [Privacy Statement](/github/site-policy/github-privacy-statement#our-use-of-cookies-and-tracking-technologies) if you’d like more information about cookies, and on how and why we use them and cookie-related personal data. You can change your preference about non-essential cookies at any time by following [these instructions](/account-and-profile/setting-up-and-managing-your-personal-account-on-github/managing-personal-account-settings/managing-your-cookie-preferences-for-githubs-enterprise-marketing-pages). | |
| GitHub uses cookies to provide and secure our websites, as well as to analyze the usage of our websites, in order to offer you a great user experience. Please take a look at our [Privacy Statement](/site-policy/privacy-policies/github-privacy-statement#our-use-of-cookies-and-tracking-technologies) if you’d like more information about cookies, and on how and why we use them and cookie-related personal data. You can change your preference about non-essential cookies at any time by following [these instructions](/account-and-profile/setting-up-and-managing-your-personal-account-on-github/managing-personal-account-settings/managing-your-cookie-preferences-for-githubs-enterprise-marketing-pages). |
| [Google](https://policies.google.com/privacy) | `SAPISID` | This cookie is used to build a profile of the website visitor's interests and show relevant ads on other sites. | 90 Days | ||
| [Google](https://policies.google.com/privacy) | `test_cookie` | This cookie is used to determine if the website visitor's browser supports cookies. | 90 Days | ||
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | `bcookie` | This cookie is a browser identifier cookie to uniquely identify devices accessing LinkedIn to detect abuse on the platform. | One year | ||
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | bscookie | This cookie is used for remembering that a logged in user is verified by two factor authentication. | One year |
There was a problem hiding this comment.
The cookie name "bscookie" is missing backticks for consistent formatting. All other cookie names in the table are properly formatted with backticks.
Suggested change
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | bscookie | This cookie is used for remembering that a logged in user is verified by two factor authentication. | One year | |
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | `bscookie` | This cookie is used for remembering that a logged in user is verified by two factor authentication. | One year |
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | `li_fat_id` | This cookie is used for conversion tracking, retargeting, analytics. | 30 Days | ||
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | `li_sugr` | This cookie is used to make a probabilistic match of a user's identity. | 90 Days | ||
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | `U` | This cookie is used as a browser identifier. | 3 months | ||
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | BizographicsOptOutBizographicsOptOut | This cookie is used to determine opt-out status for non-members. | 10 years |
There was a problem hiding this comment.
The cookie name "BizographicsOptOutBizographicsOptOut" is missing backticks for consistent formatting. All other cookie names in the table should be properly formatted with backticks.
Suggested change
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | BizographicsOptOutBizographicsOptOut | This cookie is used to determine opt-out status for non-members. | 10 years | |
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | `BizographicsOptOutBizographicsOptOut` | This cookie is used to determine opt-out status for non-members. | 10 years |
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | `li_sugr` | This cookie is used to make a probabilistic match of a user's identity. | 90 Days | ||
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | `U` | This cookie is used as a browser identifier. | 3 months | ||
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | BizographicsOptOutBizographicsOptOut | This cookie is used to determine opt-out status for non-members. | 10 years | ||
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | `li_giant` | This cookie is used for conversion tracking. | 7 Days | https://www.linkedin.com/legal/privacy-policy |
There was a problem hiding this comment.
This row has an extra column with a duplicate privacy policy link that doesn't match the table structure. The table should have only 4 columns: Provider, Cookie Name, Description, and Expiration.
Suggested change
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | `li_giant` | This cookie is used for conversion tracking. | 7 Days | https://www.linkedin.com/legal/privacy-policy | |
| [LinkedIn](https://www.linkedin.com/legal/privacy-policy) | `li_giant` | This cookie is used for conversion tracking. | 7 Days |
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.
repo sync