docs: Include tips for running alongside Test Suite when maintaining · openactive/OpenActive.Server.NET@237f2b2

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

The 'PackageIconUrl'/'iconUrl' element is deprecated. Consider using the 'PackageIcon'/'icon' element instead. Learn more at https://aka.ms/deprecateIconUrl

OpenActive.Server.NET/CustomBookingEngine/CustomBookingEngine.cs#L154

This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.

OpenActive.Server.NET/StoreBookingEngine/StoreBookingEngine.cs#L38

Avoid empty interfaces

OpenActive.Server.NET/StoreBookingEngine/StoreBookingEngine.cs#L266

Exceptions should be public

OpenActive.Server.NET/StoreBookingEngine/StoreBookingEngine.cs#L266

Add the following constructor to SilentRollbackException: public SilentRollbackException(string message).

OpenActive.Server.NET/StoreBookingEngine/StoreBookingEngine.cs#L266

Add the following constructor to SilentRollbackException: public SilentRollbackException(string message, Exception innerException).

OpenActive.Server.NET/StoreBookingEngine/StoreBookingEngine.cs#L365

Consider calling ConfigureAwait on the awaited task

OpenActive.Server.NET/StoreBookingEngine/StoreBookingEngine.cs#L373

Consider calling ConfigureAwait on the awaited task