Cleanup code warnings in Rider and SonarLint

SharpShuffleBag.Tests/FixedSequenceSourceTests.cs

@@ -1,6 +1,6 @@
 namespace SharpShuffleBag.Tests;
 
-public class FixedSequenceSourceTests
+public sealed class FixedSequenceSourceTests
 {
 	[Fact]
 	public void SmokeTest()

SharpShuffleBag.Unity/Assets/Editor/SampleDevelopment.cs

@@ -22,8 +22,8 @@ public static void CopySamplesToPackage()
 			{
 				foreach (DirectoryInfo source in versionDirectory.EnumerateDirectories())
 				{
-					// Packages/com.xarbrough.sharp-shuffle-bag-unity/Samples~/Example1
-					// Packages/com.xarbrough.sharp-shuffle-bag-unity/Samples~/Example2
+					// Packages/com.chrisyarbrough.sharpshufflebag/Samples~/Example1
+					// Packages/com.chrisyarbrough.sharpshufflebag/Samples~/Example2
 					string dest = "Packages/com.chrisyarbrough.sharpshufflebag/Samples~/" + source.Name;
 					CopyDirectory(source, dest);
 				}

SharpShuffleBag.Unity/Packages/com.chrisyarbrough.sharpshufflebag/Core/IRandomRangeSource.cs

@@ -1,3 +1,5 @@
+// ReSharper disable UnusedMember.Global
+
 namespace SharpShuffleBag
 {
 	using System;

SharpShuffleBag.Unity/Packages/com.chrisyarbrough.sharpshufflebag/Core/IRandomValueSource.cs

@@ -1,3 +1,5 @@
+// ReSharper disable UnusedMember.Global
+
 namespace SharpShuffleBag
 {
 	/// <summary>

SharpShuffleBag.Unity/Packages/com.chrisyarbrough.sharpshufflebag/Core/ShuffleBag.cs

@@ -1,4 +1,6 @@
-namespace SharpShuffleBag
+// ReSharper disable UnusedMember.Global
+
+namespace SharpShuffleBag
 {
 	using System;
 	using System.Collections;

SharpShuffleBag.Unity/Packages/com.chrisyarbrough.sharpshufflebag/Core/ShuffleBagInt.cs

@@ -1,3 +1,5 @@
+// ReSharper disable UnusedMember.Global
+
 namespace SharpShuffleBag
 {
 	using System.Collections.Generic;

SharpShuffleBag.Unity/Packages/com.chrisyarbrough.sharpshufflebag/README.md

@@ -57,3 +57,4 @@ and therefore avoids the larger performance spike of reshuffling/refilling the b
 - Efficient implementation: The underlying collection is not resized, instead,
   an index is used to perform the random selection.
 - Set a custom `RandomSource` for fine-grained control (e.g. to make the bag deterministic or set a seed).
+- `Shuffle.FisherYates(IList<T> list)` utility method to shuffle a list in-place.

SharpShuffleBag.Unity/Packages/com.chrisyarbrough.sharpshufflebag/Samples~/Example/RandomSpawner.cs

@@ -21,19 +21,23 @@ internal sealed class RandomSpawner : MonoBehaviour
 		[SerializeField]
 		private float spawnInterval = 0.5f;
 
-		private ShuffleBag<FadingTextMesh> shuffleBag;
-
 		private IEnumerator Start()
 		{
 			Application.targetFrameRate = 60;
 
-			shuffleBag = new ShuffleBag<FadingTextMesh>(SpawnTexts());
+			ShuffleBag<FadingTextMesh> shuffleBag = new(SpawnTexts());
 
+			int i = 0;
 			while (Application.isPlaying)
 			{
 				yield return new WaitForSeconds(spawnInterval);
+
 				FadingTextMesh text = shuffleBag.Next();
 				text.StartFade();
+
+				if (i++ % shuffleBag.Size == 0)
+					Debug.Log("----");
+				Debug.Log(text.Text);
 			}
 		}
 

SharpShuffleBag.Unity/Packages/com.chrisyarbrough.sharpshufflebag/Unity/DefaultRandomSourceInitializer.cs

@@ -10,7 +10,7 @@ internal static class DefaultRandomSourceInitializer
 		[RuntimeInitializeOnLoadMethod(RuntimeInitializeLoadType.AfterAssembliesLoaded)]
 		private static void Initialize()
 		{
-			if (IRandomRangeSource.Default.GetType() != typeof(UnityRandomSource))
+			if (IRandomRangeSource.Default is UnityRandomSource)
 			{
 				IRandomRangeSource.Default = new UnityRandomSource();
 			}

SharpShuffleBag/IRandomRangeSource.cs

@@ -1,3 +1,5 @@
+// ReSharper disable UnusedMember.Global
+
 namespace SharpShuffleBag
 {
 	using System;

SharpShuffleBag/IRandomValueSource.cs

@@ -1,3 +1,5 @@
+// ReSharper disable UnusedMember.Global
+
 namespace SharpShuffleBag
 {
 	/// <summary>

SharpShuffleBag/ShuffleBag.cs

@@ -1,4 +1,6 @@
-namespace SharpShuffleBag
+// ReSharper disable UnusedMember.Global
+
+namespace SharpShuffleBag
 {
 	using System;
 	using System.Collections;

SharpShuffleBag/ShuffleBagInt.cs

@@ -1,3 +1,5 @@
+// ReSharper disable UnusedMember.Global
+
 namespace SharpShuffleBag
 {
 	using System.Collections.Generic;