Skip to content

Various code cleanups in preparation of auto-formatting the codebase #960

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

Merged
merged 61 commits into from
Mar 3, 2021

Conversation

bart-degreed
Copy link
Contributor

No description provided.

Bart Koelman added 30 commits February 15, 2021 11:29
Resharper was set to auto-detect, making it ignore configured rules
…955)

Corrected link rendering for hosting inside an IIS application virtual directory
Bart Koelman added 24 commits February 25, 2021 12:29
…ike 'member can be private, set accessor is never used'etc.
…nly used for precondition checks'

Note that one suppression is needed, due to a bug in Resharper
…ause these are assigned once at application startup by the resource graph builder and then are considered immutable.
…mber can be made internal', 'type has no inheritors and can be marked sealed', 'member is never written to' etc.
…r an open generic for them to enable different implementations per resource type.
…ld find the line that calls Subject.StatusCode.Should().Be() and treat the Subject variable as the subject-under-test (SUT). But by applying this attribute, it will ignore this invocation and instead find the SUT by looking for a call to Should().HaveStatusCode() and use the response variable instead.
@bart-degreed bart-degreed requested a review from maurei March 3, 2021 10:54
@bart-degreed bart-degreed merged commit 745d2fb into master Mar 3, 2021
@bart-degreed bart-degreed deleted the coding-guidelines-4 branch March 3, 2021 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants