📝 Standard Folder FWLink List #3429

xaviex · 2017-10-17T00:38:22Z

FWLink Standard Folder fix

Summary

FWLink Standard folder update to clarify links in the /docs/standard folder

Suggested Reviewers

mairaw

Looks great @xaviex! Really appreciate you tackling this. Again, please let me know if you'd like me to address the changes I'm requesting.

mairaw · 2017-10-17T00:40:54Z

docs/standard/base-types/best-practices.md

 -   When developing a pattern, you should consider how backtracking might affect the performance of the regular expression engine, particularly if your regular expression is designed to process unconstrained input. For more information, see the [Take Charge of Backtracking](#Backtracking) section.  

-   Thoroughly test your regular expression using invalid and near-valid input as well as valid input. To generate input for a particular regular expression randomly, you can use [Rex](http://go.microsoft.com/fwlink/?LinkId=210756), which is a regular expression exploration tool from Microsoft Research.  
+-   Thoroughly test your regular expression using invalid and near-valid input as well as valid input. To generate input for a particular regular expression randomly, you can use [Rex](http://research.microsoft.com/en-us/projects/rex/), which is a regular expression exploration tool from Microsoft Research.  


It seems the new URL is https://www.microsoft.com/en-us/research/project/rex-regular-expression-exploration/

mairaw · 2017-10-17T00:42:08Z

docs/standard/base-types/best-practices.md


 > [!NOTE]
->  For a more detailed discussion of the performance implications of using interpreted and compiled regular expressions, see [Optimizing Regular Expression Performance, Part II: Taking Charge of Backtracking](http://go.microsoft.com/fwlink/?LinkId=211566) in the BCL Team blog.  
+>  For a more detailed discussion of the performance implications of using interpreted and compiled regular expressions, see [Optimizing Regular Expression Performance, Part II: Taking Charge of Backtracking](http://blogs.msdn.com/b/bclteam/archive/2010/06/25/optimizing-regular-expression-performance-part-i-working-with-the-regex-class-and-regex-objects.aspx) in the BCL Team blog.  


it seems the right url would be https://blogs.msdn.microsoft.com/bclteam/2010/08/03/optimizing-regular-expression-performance-part-ii-taking-charge-of-backtracking-ron-petrusha/

It seems even the forward link goes to the incorrect location, would we like this to be changed to the proper location or no?

yes, please change to https://blogs.msdn.microsoft.com/bclteam/2010/08/03/optimizing-regular-expression-performance-part-ii-taking-charge-of-backtracking-ron-petrusha/

mairaw · 2017-10-17T00:43:21Z

docs/standard/base-types/best-practices.md


 > [!NOTE]
->  For more information on backtracking, see [Details of Regular Expression Behavior](../../../docs/standard/base-types/details-of-regular-expression-behavior.md) and [Backtracking](../../../docs/standard/base-types/backtracking-in-regular-expressions.md). For a detailed discussion of backtracking, see [Optimizing Regular Expression Performance, Part II: Taking Charge of Backtracking](http://go.microsoft.com/fwlink/?LinkId=211567) in the BCL Team blog.  
+>  For more information on backtracking, see [Details of Regular Expression Behavior](../../../docs/standard/base-types/details-of-regular-expression-behavior.md) and [Backtracking](../../../docs/standard/base-types/backtracking-in-regular-expressions.md). For a detailed discussion of backtracking, see [Optimizing Regular Expression Performance, Part II: Taking Charge of Backtracking](http://blogs.msdn.com/b/bclteam/archive/2010/08/03/optimizing-regular-expression-performance-part-ii-taking-charge-of-backtracking.aspx) in the BCL Team blog.  


same here. final url is https://blogs.msdn.microsoft.com/bclteam/2010/08/03/optimizing-regular-expression-performance-part-ii-taking-charge-of-backtracking-ron-petrusha/

mairaw · 2017-10-17T00:44:34Z

docs/standard/cross-platform/cross-platform-development-with-the-portable-class-library.md

 |Visual Studio SKU|Support for creating a Portable Class Library|  
 |-----------------------|---------------------------------------------------|  
-|Visual Studio 2010, Professional, Premium, or Ultimate|Yes, when you install the [Portable Library Tools](http://go.microsoft.com/fwlink/?LinkId=210823).|  
+|Visual Studio 2010, Professional, Premium, or Ultimate|Yes, when you install the [Portable Library Tools](http://visualstudiogallery.msdn.microsoft.com/b0e0b5e9-e138-410b-ad10-00cb3caf4981/).|  


final URL is https://marketplace.visualstudio.com/items?itemName=BCLTeam.PortableLibraryTools2

mairaw · 2017-10-17T00:47:21Z

docs/standard/cross-platform/cross-platform-development-with-the-portable-class-library.md

 For example, the Portable Class Library contains UI-related types only when you target Windows 8.1 and Windows Phone 8.1. Also, you may encounter limitations if you target platforms (such as Xbox, the .NET Framework 4, and Windows Phone 7) that were released before the introduction of the Portable Class Library. The .NET Framework releases packages through NuGet that improves the Portable Class Library support for some of these older platforms. For more information and a list of NuGet packages, see [The .NET Framework and Out-of-Band Releases](../../../docs/framework/get-started/the-net-framework-and-out-of-band-releases.md).  

- If a member is supported in the Portable Class Library and for your selected targets, it will appear in your project in IntelliSense. In addition, the Portable Class Library icon ![Supported by Portable Library](../../../docs/standard/cross-platform/media/portablelibrary-referenceicon.png "PortableLibrary_ReferenceIcon") appears in the members tables, in the [.NET Framework Class Library](http://go.microsoft.com/fwlink/?LinkId=211358) next to supported members. For example, the following members table shows that the <xref:System.String.Chars%2A> property in the <xref:System.String> class is supported in the Portable Class Library:  
+ If a member is supported in the Portable Class Library and for your selected targets, it will appear in your project in IntelliSense. In addition, the Portable Class Library icon ![Supported by Portable Library](../../../docs/standard/cross-platform/media/portablelibrary-referenceicon.png "PortableLibrary_ReferenceIcon") appears in the members tables, in the [.NET Framework Class Library](http://msdn.microsoft.com/en-us/library/ms229335.aspx) next to supported members. For example, the following members table shows that the <xref:System.String.Chars%2A> property in the <xref:System.String> class is supported in the Portable Class Library:  


right link would be https://msdn.microsoft.com/library/mt472912.aspx. I just updated in the tool too in case other pages are using this link

mairaw · 2017-10-17T01:01:26Z

docs/standard/io/index.md

 Isolated storage is a data storage mechanism that provides isolation and safety by defining standardized ways of associating code with saved data. The storage provides a virtual file system that is isolated by user, assembly, and (optionally) domain. Isolated storage is particularly useful when your application does not have permission to access user files. You can save settings or files for your application in a manner that is controlled by the computer's security policy.  

- Isolated storage is not available for [!INCLUDE[win8_appname_long](../../../includes/win8-appname-long-md.md)] apps; instead, use application data classes in the [Windows.Storage](http://msdn.microsoft.com/library/windows/apps/windows.storage.aspx) namespace. For more information, see [Application data](http://go.microsoft.com/fwlink/?LinkId=229175) in the Windows Dev Center.  
+ Isolated storage is not available for [!INCLUDE[win8_appname_long](../../../includes/win8-appname-long-md.md)] apps; instead, use application data classes in the [Windows.Storage](http://msdn.microsoft.com/library/windows/apps/windows.storage.aspx) namespace. For more information, see [Application data](http://msdn.microsoft.com/en-us/library/windows/apps/hh464917.aspx) in the Windows Dev Center.  


remove en-us/ from the link

mairaw · 2017-10-17T01:01:57Z

docs/standard/io/index.md

 -   Types specifically related to file operations, such as <xref:System.IO.File>, <xref:System.IO.FileInfo>, <xref:System.IO.Directory> and <xref:System.IO.DirectoryInfo>, are not included in the [!INCLUDE[net_win8_profile](../../../includes/net-win8-profile-md.md)]. Instead, use the types in the [Windows.Storage](http://msdn.microsoft.com/library/windows/apps/windows.storage.aspx) namespace of the [!INCLUDE[wrt](../../../includes/wrt-md.md)], such as [StorageFile](http://msdn.microsoft.com/library/windows/apps/windows.storage.storagefile.aspx) and [StorageFolder](http://msdn.microsoft.com/library/windows/apps/windows.storage.storagefolder.aspx).  

-   Isolated storage is not available; instead, use [application data](http://go.microsoft.com/fwlink/?LinkId=229175).  
+-   Isolated storage is not available; instead, use [application data](http://msdn.microsoft.com/en-us/library/windows/apps/hh464917.aspx).  


remove en-us/ from the link

mairaw · 2017-10-17T01:02:16Z

docs/standard/io/index.md

 When you use the classes in the <xref:System.IO?displayProperty=nameWithType> namespace, you must follow operating system security requirements such as access control lists (ACLs) to control access to files and directories. This requirement is in addition to any <xref:System.Security.Permissions.FileIOPermission> requirements. You can manage ACLs programmatically. For more information, see [How to: Add or Remove Access Control List Entries](../../../docs/standard/io/how-to-add-or-remove-access-control-list-entries.md).  

- Default security policies prevent Internet or intranet applications from accessing files on the user’s computer. Therefore, do not use the I/O classes that require a path to a physical file when writing code that will be downloaded over the Internet or intranet. Instead, use [isolated storage](../../../docs/standard/io/isolated-storage.md) for traditional .NET Framework applications, or use [application data](http://go.microsoft.com/fwlink/?LinkId=229175) for [!INCLUDE[win8_appname_long](../../../includes/win8-appname-long-md.md)] apps.  
+ Default security policies prevent Internet or intranet applications from accessing files on the user’s computer. Therefore, do not use the I/O classes that require a path to a physical file when writing code that will be downloaded over the Internet or intranet. Instead, use [isolated storage](../../../docs/standard/io/isolated-storage.md) for traditional .NET Framework applications, or use [application data](http://msdn.microsoft.com/en-us/library/windows/apps/hh464917.aspx) for [!INCLUDE[win8_appname_long](../../../includes/win8-appname-long-md.md)] apps.  


remove en-us from the link

mairaw · 2017-10-17T01:02:37Z

docs/standard/io/isolated-storage.md


 > [!IMPORTANT]
->  Isolated storage is not available for [!INCLUDE[win8_appname_long](../../../includes/win8-appname-long-md.md)] apps. Instead, use the application data classes in the `Windows.Storage` namespaces included in the [!INCLUDE[wrt](../../../includes/wrt-md.md)] API to store local data and files. For more information, see [Application data](http://go.microsoft.com/fwlink/?LinkId=229175) in the Windows Dev Center.  
+>  Isolated storage is not available for [!INCLUDE[win8_appname_long](../../../includes/win8-appname-long-md.md)] apps. Instead, use the application data classes in the `Windows.Storage` namespaces included in the [!INCLUDE[wrt](../../../includes/wrt-md.md)] API to store local data and files. For more information, see [Application data](http://msdn.microsoft.com/en-us/library/windows/apps/hh464917.aspx) in the Windows Dev Center.  


remove en-us from the link

mairaw · 2017-10-17T01:02:46Z

docs/standard/io/types-of-isolation.md


 > [!IMPORTANT]
->  Isolated storage is not available for [!INCLUDE[win8_appname_long](../../../includes/win8-appname-long-md.md)] apps. Instead, use the application data classes in the `Windows.Storage` namespaces included in the [!INCLUDE[wrt](../../../includes/wrt-md.md)] API to store local data and files. For more information, see [Application data](http://go.microsoft.com/fwlink/?LinkId=229175) in the Windows Dev Center.  
+>  Isolated storage is not available for [!INCLUDE[win8_appname_long](../../../includes/win8-appname-long-md.md)] apps. Instead, use the application data classes in the `Windows.Storage` namespaces included in the [!INCLUDE[wrt](../../../includes/wrt-md.md)] API to store local data and files. For more information, see [Application data](http://msdn.microsoft.com/en-us/library/windows/apps/hh464917.aspx) in the Windows Dev Center.  


This reverts commit 530b057.

mairaw · 2017-10-17T19:03:06Z

docs/standard/base-types/best-practices.md

a more cleaner url is https://www.microsoft.com/en-us/research/project/rex-regular-expression-exploration/

mairaw · 2017-10-17T19:04:15Z

docs/standard/base-types/best-practices.md


 > [!NOTE]
->  For a more detailed discussion of the performance implications of using interpreted and compiled regular expressions, see [Optimizing Regular Expression Performance, Part II: Taking Charge of Backtracking](http://go.microsoft.com/fwlink/?LinkId=211566) in the BCL Team blog.  
+>  For a more detailed discussion of the performance implications of using interpreted and compiled regular expressions, see [Optimizing Regular Expression Performance, Part II: Taking Charge of Backtracking](http://blogs.msdn.com/b/bclteam/archive/2010/06/25/optimizing-regular-expression-performance-part-i-working-with-the-regex-class-and-regex-objects.aspx) in the BCL Team blog.  


yes, please change to https://blogs.msdn.microsoft.com/bclteam/2010/08/03/optimizing-regular-expression-performance-part-ii-taking-charge-of-backtracking-ron-petrusha/

xaviex · 2017-10-17T21:35:33Z

@mairaw I have cleaned up that odd-looking link, where there any other discrepancies?

xaviex · 2017-10-17T23:08:08Z

It has come to my attention that this has actually removed more localization than was in the link replacements, is this ok?

mairaw

Thanks for the fixes. The global changes to remove en-us from the links generated a new set of feedback @xaviex. In general, I'd recommend we solve one issue per PR. Let me know if you need help addressing the comments. Thanks again!

mairaw · 2017-10-17T22:56:46Z

docs/standard/base-types/best-practices.md

Let's change this to https://blogs.msdn.microsoft.com/bclteam/2010/08/03/optimizing-regular-expression-performance-part-ii-taking-charge-of-backtracking-ron-petrusha/

mairaw · 2017-10-17T22:59:54Z

...fecycle-architecture/design-develop-containerized-apps/common-container-design-principles.md

WebJobs link now actually goes here: https://github.com/Azure/azure-webjobs-sdk/wiki

mairaw · 2017-10-17T23:01:12Z

...inerized-lifecycle-architecture/design-develop-containerized-apps/monolithic-applications.md

The scale sets link now goes here: https://docs.microsoft.com/azure/virtual-machine-scale-sets/ (and in the case of the e-books we keep the docs URL)

mairaw · 2017-10-17T23:07:34Z

docs/standard/containerized-lifecycle-architecture/docker-terminology.md

let's update this link to https://www.microsoft.com/cloud-platform/server-virtualization

mairaw · 2017-10-17T23:09:09Z

docs/standard/containerized-lifecycle-architecture/what-is-docker.md

let's update this link to https://docs.microsoft.com/virtualization/windowscontainers/about/index

mairaw · 2017-10-17T23:34:11Z

...microservices-architecture/multi-container-microservice-net-applications/subscribe-events.md

change this to https://docs.microsoft.com/azure/architecture/patterns/event-sourcing

mairaw · 2017-10-17T23:36:06Z

...microservices-architecture/multi-container-microservice-net-applications/subscribe-events.md

@BillWagner this link doesn't seem to exist. delete?

mairaw · 2017-10-17T23:36:40Z

...itecture/multi-container-microservice-net-applications/test-aspnet-core-services-web-apps.md

change this to https://docs.microsoft.com/visualstudio/test/unit-test-basics

mairaw · 2017-10-17T23:39:58Z

docs/standard/parallel-programming/how-to-speed-up-small-loop-bodies.md

change For link to ~/docs/visual-basic/language-reference/statements/for-each-next-statement.md

mairaw · 2017-10-17T23:44:47Z

docs/standard/threading/threads-and-threading.md

nit: there's a missing space before the link if you'd like to fix that

xaviex · 2017-10-18T01:24:38Z

the removal of en-us was actually accidental on my part, I can re do it only changing the links that used to be fwlinks?

xaviex · 2017-10-18T01:26:39Z

@mariaw Would you like me to create a new pull request with the basic changes along with what you have commented on here?

mairaw · 2017-10-19T02:29:20Z

No need to open a new PR. We could probably just pull out the commit you made that change from this PR. I'll do that.

xaviex · 2017-10-19T21:58:17Z

@mairaw Was that change made I can not tell?

mairaw · 2017-10-20T07:57:22Z

I've removed that commit now @xaviex.

mairaw · 2017-11-06T21:43:38Z

@xaviex do you want to take care of the remainder comments or do you want me to address them? Sorry this PR has been lingering here but I was out for the past couple of weeks.

xaviex · 2017-11-07T22:26:23Z

@mairaw Ive got this, should be pushing it in a day or two

mairaw · 2017-11-09T00:56:17Z

Thank you @xaviex!

BillWagner · 2017-11-20T21:43:05Z

@xaviex Are you still planning to make those final changes? If not, we'll address them and merge your PR.

Thanks again for contributing.

xaviex · 2017-11-20T22:13:20Z

Im still planning on it, i just had something come up in the last weeks so the code has lain dormant, i will be able to get it up soon

…

On Mon, Nov 20, 2017 at 4:43 PM Bill Wagner ***@***.***> wrote: @xaviex <https://github.com/xaviex> Are you still planning to make those final changes? If not, we'll address them and merge your PR. Thanks again for contributing. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3429 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABVYXnF7NvoRNPuGLj2gfZQe5Y72PR6kks5s4fJygaJpZM4P7bIl> .

xaviex · 2017-12-02T01:31:39Z

@mairaw Hey my apologies, it doesn't look like I'm going to be able to get to this within the next week or so, work got very hectic recently, thanks for all the help

mairaw · 2017-12-02T02:55:35Z

No worries @xaviex. I'll see what the remaining comments were and finish this PR. Thanks a bunch!

Standard Folder FWLink List

530b057

dnfclas added the cla-signed label Oct 17, 2017

mairaw requested changes Oct 17, 2017

View reviewed changes

xaviex added 5 commits October 17, 2017 10:57

Revert "Standard Folder FWLink List"

1e405cc

This reverts commit 530b057.

FWLinks

2fe8892

Missed one

2aea3b4

More FWLink Replacements

31cdf52

Final FWLinks

8b21a97

mairaw added the ✨ 1st-time docs contributor! Indicates PRs from new contributors to the docs repository label Oct 17, 2017

mairaw reviewed Oct 17, 2017

View reviewed changes

mairaw removed the ✨ 1st-time docs contributor! Indicates PRs from new contributors to the docs repository label Oct 17, 2017

mairaw requested changes Oct 17, 2017

View reviewed changes

mairaw changed the title ~~Standard Folder FWLink List~~ 📝 Standard Folder FWLink List Oct 18, 2017

Fixed Odd Looking Link

3fdf272

mairaw force-pushed the StandardFWLinks branch from f2edf0e to 3fdf272 Compare October 20, 2017 07:56

jongalloway removed the cla-signed label Oct 31, 2017

dotnet deleted a comment from dnfclas Oct 31, 2017

Merge branch 'master' into StandardFWLinks

57e4520

Merge branch 'master' into StandardFWLinks

8e80846

applying feedback

aa914cd

mairaw added the verify-build-before-merge label Jan 8, 2018

mairaw approved these changes Jan 9, 2018

View reviewed changes

mairaw merged commit 4ddadd3 into dotnet:master Jan 9, 2018

mairaw removed the verify-build-before-merge label Jan 9, 2018

📝 Standard Folder FWLink List #3429

📝 Standard Folder FWLink List #3429

Uh oh!

Conversation

xaviex commented Oct 17, 2017

FWLink Standard Folder fix

Summary

Suggested Reviewers

Uh oh!

mairaw left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

xaviex commented Oct 17, 2017

Uh oh!

xaviex commented Oct 17, 2017

Uh oh!

mairaw left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

xaviex commented Oct 18, 2017

Uh oh!

xaviex commented Oct 18, 2017

Uh oh!

mairaw commented Oct 19, 2017

Uh oh!

xaviex commented Oct 19, 2017

Uh oh!

mairaw commented Oct 20, 2017

Uh oh!

mairaw commented Nov 6, 2017

Uh oh!

xaviex commented Nov 7, 2017