Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Microsoft.CodeAnalysis.PublicApiAnalyzers #1447

Merged
merged 1 commit into from
Dec 19, 2023

Conversation

lukebakken
Copy link
Contributor

Reference: #1269

Use the same API analysis tool that is used in the RabbitMQ.Stream.Client library.

@lukebakken lukebakken added this to the 7.0.0 milestone Dec 18, 2023
@lukebakken lukebakken self-assigned this Dec 18, 2023
@lukebakken lukebakken force-pushed the rabbitmq-dotnet-client-1269 branch from 59d86ae to 1fa2325 Compare December 19, 2023 16:15
@lukebakken lukebakken marked this pull request as ready for review December 19, 2023 17:37
Reference: #1269

* Use the same API analysis tool that is used in the RabbitMQ.Stream.Client library.
* Remove existing APIApproval tests. Replaced by PublicApiAnalyzers.
* Add retries when connection can't be open due to a `SocketException`.
* Change Windows GitHub actions runner to windows-2019. It should be faster and may not have that socket exception
* Add connection retries for SocketException to SSL tests. Uncovered with windows 2019.
@lukebakken lukebakken force-pushed the rabbitmq-dotnet-client-1269 branch from 1fa2325 to a0511ad Compare December 19, 2023 17:43
@lukebakken lukebakken merged commit 16d1744 into main Dec 19, 2023
10 checks passed
@lukebakken lukebakken deleted the rabbitmq-dotnet-client-1269 branch December 19, 2023 19:10
lukebakken added a commit that referenced this pull request Dec 19, 2023
lukebakken added a commit that referenced this pull request Dec 20, 2023
Follow-up to #1447

Ensure CreateConnectionFactory is used so that client name is provided

Look for either of two exceptions
lukebakken added a commit that referenced this pull request Dec 20, 2023
Follow-up to #1447

Ensure CreateConnectionFactory is used so that client name is provided

Look for either of two exceptions
lukebakken added a commit that referenced this pull request Dec 20, 2023
Follow-up to #1447

Ensure CreateConnectionFactory is used so that client name is provided

Look for either of two exceptions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant