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.

Sign up for GitHub

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

Jump to bottom

Roslyn Analyzer must report System.DateTimeOffset.Now|UtcNow #328

Open
allantargino opened this issue Jun 10, 2024 · 1 comment
Open

Roslyn Analyzer must report System.DateTimeOffset.Now|UtcNow #328

allantargino opened this issue Jun 10, 2024 · 1 comment
Labels
Analyzer Enhancement New feature or request P2

Comments

@allantargino
Copy link
Member

allantargino commented Jun 10, 2024 •
edited
Loading

Our Roslyn analyzer already detects the usage of DateTime.Now, DateTime.UtcNow and DateTime.Today.
We should also report about DateTimeOffset.Now and DateTimeOffset.UtcNow.
@allantargino
Copy link
Member Author

As part of this implementation, we could investigate how to use the same idea from Banned APIs Analyzers project, which is to use a configurable/generic txt file instead of manually looking for symbols.

@lilyjma lilyjma added Enhancement New feature or request P2 labels Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Analyzer Enhancement New feature or request P2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@allantargino @lilyjma