- 
                Notifications
    You must be signed in to change notification settings 
- Fork 6.1k
43). Add topic files 135 to 138 #18826
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
43). Add topic files 135 to 138 #18826
Conversation
102eac8    to
    c652822      
    Compare
  
    | Last set of merge conflicts to resolve for now @jameshkramer 😄 | 
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.
Looks good. A few suggestions...
        
          
                docs/standard/linq/use-annotations-transform-linq-xml-trees-xslt-style.md
              
                Outdated
          
            Show resolved
            Hide resolved
        
      Co-authored-by: Maira Wenzel <[email protected]>
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.
LGTM
| @mairaw @BillWagner @tfosmark @marteeleigh Maira, thanks for getting these PRs merged. I'll have some changes to redirections this week. Then we test? Jim | 
| Sounds good @jameshkramer. There are also some warnings showing up in #16969 after I merged the last PR, so you should take a look at that as well. | 
This PR is for user story #1576037.
It adds four article files, makes corresponding deletions from the C# and VB folders, and adds redirection entries. It also fixes some article files.
Please merge #18439, #18541, #18580, #18589, #18673, #18700, #18711, #18781, and #18802 before merging this.
cc: @BillWagner @mairaw @tfosmark @marteeleigh
Contributes to #4728