Add support for react-icons

[NFNChris]

The popular react-icons module provides FontAwesome icons (among others) in best practice SVG format. I have put together a pull request that provides optional support for this library, which at the time of writing has almost 350k weekly downloads.

[NFNChris]

It seems listing "react-icons" as an "optionalDependency" in package.json results in an attempt to install it by default. I guess the alternative would be to simply leave it out of package.json entirely.

[NFNChris]

This is building locally for me without issue. It seems the TeamCity build failure is unrelated to the code and something to do with the build environment:

[15:26:11][Step 1/1] System.InvalidOperationException: Could not find a 'develop' or 'master' branch, neither locally nor remotely. [15:26:11][Step 1/1] at GitVersion.BranchConfigurationCalculator.InheritBranchConfiguration (System.Boolean onlyEvaluateTrackedBranches, LibGit2Sharp.IRepository repository, LibGit2Sharp.Commit currentCommit, LibGit2Sharp.Branch currentBranch, System.Collections.Generic.KeyValuePair2[TKey,TValue] keyValuePair, GitVersion.BranchConfig branchConfiguration, GitVersion.Config config, System.Collections.Generic.IList1[T] excludedInheritBranches) [0x00229] in <57cedcc684654481ace045b6d7bcc156>:0 [15:26:11][Step 1/1] at GitVersion.BranchConfigurationCalculator.GetBranchConfiguration (LibGit2Sharp.Commit currentCommit, LibGit2Sharp.IRepository repository, System.Boolean onlyEvaluateTrackedBranches, GitVersion.Config config, LibGit2Sharp.Branch currentBranch, System.Collections.Generic.IList1[T] excludedInheritBranches) [0x00072] in <57cedcc684654481ace045b6d7bcc156>:0 [15:26:11][Step 1/1] at GitVersion.GitVersionContext.CalculateEffectiveConfiguration () [0x0001e] in <57cedcc684654481ace045b6d7bcc156>:0 [15:26:11][Step 1/1] at GitVersion.GitVersionContext..ctor (LibGit2Sharp.IRepository repository, LibGit2Sharp.Branch currentBranch, GitVersion.Config configuration, System.Boolean onlyEvaluateTrackedBranches, System.String commitId) [0x00110] in <57cedcc684654481ace045b6d7bcc156>:0 [15:26:11][Step 1/1] at GitVersion.GitVersionContext..ctor (LibGit2Sharp.IRepository repository, GitVersion.Config configuration, System.Boolean isForTrackingBranchOnly, System.String commitId) [0x00008] in <57cedcc684654481ace045b6d7bcc156>:0 [15:26:11][Step 1/1] at GitVersion.ExecuteCore+<>c__DisplayClass6_0.<ExecuteInternal>b__0 (LibGit2Sharp.IRepository repo) [0x00000] in <57cedcc684654481ace045b6d7bcc156>:0 [15:26:11][Step 1/1] at GitVersion.GitPreparer.WithRepository[TResult] (System.Func2[T,TResult] action) [0x0000c] in <57cedcc684654481ace045b6d7bcc156>:0 [15:26:11][Step 1/1] at GitVersion.ExecuteCore.ExecuteInternal (System.String targetBranch, System.String commitId, GitVersion.GitPreparer gitPreparer, GitVersion.IBuildServer buildServer, GitVersion.Config overrideConfig) [0x0002d] in <57cedcc684654481ace045b6d7bcc156>:0 [15:26:11][Step 1/1] at GitVersion.ExecuteCore.ExecuteGitVersion (System.String targetUrl, System.String dynamicRepositoryLocation, GitVersion.Authentication authentication, System.String targetBranch, System.Boolean noFetch, System.String workingDirectory, System.String commitId, GitVersion.Config overrideConfig) [0x000c7] in <57cedcc684654481ace045b6d7bcc156>:0 [15:26:11][Step 1/1] at GitVersion.SpecifiedArgumentRunner.Run (GitVersion.Arguments arguments, GitVersion.Helpers.IFileSystem fileSystem) [0x00074] in <57cedcc684654481ace045b6d7bcc156>:0 [15:26:11][Step 1/1] at GitVersion.Program.VerifyArgumentsAndRun () [0x00124] in <57cedcc684654481ace045b6d7bcc156>:0

[chadly]

Thanks for this!

I'm wondering if it makes sense to just do a major version bump and include react-icons as a normal dependency.