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

fix: use I prefix for interfaces #136

Merged
merged 3 commits into from
Dec 14, 2022

Conversation

ostridm
Copy link
Contributor

@ostridm ostridm commented Dec 14, 2022

closes #135

@ostridm ostridm added the Type: maintenance The issues that touch on functional checks, servicing or supporting of customers label Dec 14, 2022
@ostridm ostridm self-assigned this Dec 14, 2022
@codeclimate
Copy link

codeclimate bot commented Dec 14, 2022

Code Climate has analyzed commit 76d87bc and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 95.6% (50% is the threshold).

This pull request will bring the total coverage in the repository to 97.1%.

View more on Code Climate.

@ostridm ostridm marked this pull request as ready for review December 14, 2022 14:46
@derevnjuk derevnjuk changed the title chore(*): introduce I prefix for each interface style: use I prefix for interfaces Dec 14, 2022
@ostridm ostridm force-pushed the chore_#135/interface-naming-reconciliation branch from dfa799c to 0f08c3a Compare December 14, 2022 16:55
src/SecTester.Core/IConfigurationOptions.cs Outdated Show resolved Hide resolved
src/SecTester.Core/IConfigurationOptions.cs Outdated Show resolved Hide resolved
@derevnjuk derevnjuk added Type: technical debt Changes that remove or significantly update old unused code and/or dependencies. and removed Type: maintenance The issues that touch on functional checks, servicing or supporting of customers labels Dec 14, 2022
@derevnjuk derevnjuk changed the title style: use I prefix for interfaces fix: use I prefix for interfaces Dec 14, 2022
ostridm added a commit that referenced this pull request Dec 14, 2022
@ostridm ostridm force-pushed the chore_#135/interface-naming-reconciliation branch from 0f08c3a to 8194c82 Compare December 14, 2022 18:32
@ostridm ostridm force-pushed the chore_#135/interface-naming-reconciliation branch from 8194c82 to 76d87bc Compare December 14, 2022 18:37
@derevnjuk derevnjuk merged commit 851b522 into master Dec 14, 2022
@derevnjuk derevnjuk deleted the chore_#135/interface-naming-reconciliation branch December 14, 2022 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: technical debt Changes that remove or significantly update old unused code and/or dependencies.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Interface naming reconciliation
2 participants