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

Extension cleanup #46

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Extension cleanup #46

wants to merge 6 commits into from

Conversation

netfusion
Copy link
Contributor

When merged this pull request will:

  • Improve code quality of extensions
  • Use the latest .NET framework
  • Solve all errors, warnings and notices in VS and ReSharper

@netfusion
Cleanup
ba4658a 
ReSharper adjustments
Cleanup
Minify
Modernize
@netfusion
further improvements
c64bfb5 
reformatting
optimize
@netfusion
Removed BOM
6861fd0
@netfusion
Update target framework
403792f
@netfusion
Added C# LangVersion
c21836c 
May fix appveyor
@netfusion
Recompiled extensions
0464c13
jokoho48
jokoho48 requested changes May 17, 2020
View reviewed changes
Copy link
Contributor

@jokoho48 jokoho48 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Back in the day, we decided against using var for readability reasons.

Base automatically changed from master to main March 15, 2021 03:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jokoho48 jokoho48 jokoho48 requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@netfusion @jokoho48