update live with current master #5270
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
* Updates for 2nd Edition of Modernize existing .NET applications with Azure Cloud and Windows containers * Additional updated on Modernizing with Windows Containers * Added section on Choosing Azure compute platforms * update redirects + review This PR changed a directory name and several file names. This commit updates the redirects for those changes. Also, run acrolinx and get the scores above the merge threshold. * build fixes Remove old files that were moved. Update metadata on new files and moved files not moved with 'git mv' * respond to feedback
* move samples to samples repo So much inline code... * move samples to samples repo * finishing update content for new attributes Also, run acrolinx and update where needed. * fix build errors One will require a samples PR * respond to feedback. * add link to attributes tutotial
* unnecessary byte butter allocation In my opinion, repetitive byte buffer allocations inside while loop look unnecessary since we can reuse allocated buffer. I started to learn c# recently, so it might be wrong due to lack of my understanding. It would be nice opportunity to learn things that I missed if this proposal is incorrect. Otherwise, I believe it could improve the example. * remove unnecessary allocation for Visual Basic remove unnecessary allocation for Visual Basic as @rpetrusha suggested simply compiled & tested (the change is obvious but just in case)
* Updated next button caption * Fixed typo in TOC
It previously pointed to Roslyn only. It now points to both Roslyn and F#.
…#5244) * Added rc1 links * Fixed debian link * adding space
* update sentiment analysis for included snippets Include all code snippets from a running sample. * reference snippets for taxi ML tutorial * respond to feedback
Some update to this sample broke it such that it wouldn't compile.
Same as the namespaces issue, an update to this at some point broke it such that the sample wouldn't compile.
Forgot the file extension.
Missing space between "vista" and "or"
* 2 out of 6 complete commit * Sections 3 and 4 added * Added culture-related sections * Finalized tutorial * Fixed build warning * Link tutorial from the quickstart * Addressed nits * Addressed feedback on tutorial
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.
No description provided.