Skip to content

Repo sync for protected CLA branch #5830

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

Merged
merged 47 commits into from
Sep 7, 2020
Merged

Repo sync for protected CLA branch #5830

merged 47 commits into from
Sep 7, 2020

Conversation

ghogen
Copy link
Collaborator

@ghogen ghogen commented Sep 7, 2020

The pull request is created from master637350949984665249 to master to fix git push error for protected CLA branch

DCtheGeek and others added 30 commits September 3, 2020 14:34
…The existing analyzer implementation already mark these with category Security
…e by ID tables

- Separate out the deprecated and unported (which may be ported in future) rules into separate pages and move it under Legacy analysis section
- Remove the CA rule entries for the above bucket of rules from the top level rule table pages, both the main page that lists all CA rules and each category page that lists CA rules for that category
- Ensure that CA rule entries in main page that lists all CA IDs is in complete sync with CA IDs listed in per-category pages
- Remove the rule category pages that only have deprecated/unported rule IDs

This change should ensure:
- All the CA rule ID tables and pages under top level Reference section exactly match the CA rule IDs implemented in the analyzer assemblies that ship with .NET5 SDK
- All the deprecated/unported CA rule pages are now under Legacy analysis section

We can now bulk move all the CA rule ID tables and pages under the top level References section directly to dotnet docs repo.
Co-authored-by: Gordon Hogenson <[email protected]>
ghogen and others added 17 commits September 4, 2020 16:19
Separate out the deprecated/unported CA rules and clean up the CA rule by ID tables
Don't use VS header in Docker docset
Confirm merge from repo_sync_working_branch to master to sync with https://github.com/MicrosoftDocs/visualstudio-docs (branch master)
Add caption to one screenshot in vs-pluralsight.md
@ghogen ghogen added the CLA label Sep 7, 2020
@ghogen ghogen merged commit 01a7ebd into master Sep 7, 2020
@PRMerger6
Copy link
Contributor

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/Microsoft/azure-docs/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Microsoft/azure-docs-pr/blob/master/articles/azure-portal-overview.md
We have closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions

@fenxu fenxu deleted the master637350949984665249 branch July 16, 2021 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants