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 S2094 FP: Allow to have empty classes, used as generic parameter #6910

Closed
sm-g opened this issue Mar 12, 2023 · 0 comments · Fixed by #6926
Closed

Fix S2094 FP: Allow to have empty classes, used as generic parameter #6910

sm-g opened this issue Mar 12, 2023 · 0 comments · Fixed by #6926
Assignees
Projects
Milestone

Comments

@sm-g
Copy link

sm-g commented Mar 12, 2023

I have class

using Microsoft.AspNetCore.Authorization;

public class TokenRequirement : IAuthorizationRequirement
{
}

and later use it:

TokenRequirementHandler : AuthorizationHandler<TokenRequirement>

Note: usage could be in other csproj, i.e. for StringLocalizer we have separate csproj for Resources with defined empty classes.

@zsolt-kolbay-sonarsource zsolt-kolbay-sonarsource added this to the 8.55 milestone Mar 14, 2023
@github-actions github-actions bot added this to To do in Best Kanban Mar 14, 2023
@github-actions github-actions bot moved this from To do to Review in progress in Best Kanban Mar 14, 2023
@github-actions github-actions bot moved this from Review in progress to In progress in Best Kanban Mar 15, 2023
@github-actions github-actions bot moved this from In progress to Review approved in Best Kanban Mar 17, 2023
@csaba-sagi-sonarsource csaba-sagi-sonarsource modified the milestones: 8.55, 8.56 Mar 21, 2023
Best Kanban automation moved this from Review approved to Validate Peach Mar 21, 2023
@zsolt-kolbay-sonarsource zsolt-kolbay-sonarsource moved this from Validate Peach to Done in Best Kanban Mar 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Best Kanban
  
Done
Development

Successfully merging a pull request may close this issue.

4 participants