Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Removing unused using formats/changes other parts of the document #10237

Closed
davkean opened this issue Mar 31, 2016 · 2 comments
Closed

Removing unused using formats/changes other parts of the document #10237

davkean opened this issue Mar 31, 2016 · 2 comments
Labels
Area-IDE Bug help wanted The issue is "up for grabs" - add a comment if you are interested in working on it
Milestone

Comments

@davkean
Copy link
Member

davkean commented Mar 31, 2016

  1. Remove unused usings from the following:
Imports System

Module Module1

Sub Main()

End Sub

End Module
using System;

namespace ConsoleApplication68
{
class Program
{
}
}

Expected:

Module Module1

Sub Main()

End Sub

End Module
namespace ConsoleApplication68
{
class Program
{
}
}

Actual:

Module Module1

    Sub Main()

    End Sub

End Module
namespace ConsoleApplication68
{
    class Program
{
}
}

Make note, while in a single file this might seem that bad - when applying this across a significant project, the Preview window shows very strange diffs like below that caused me not to trust it as it's not obvious what's changing:

image

@Pilchie Pilchie added this to the 2.0 (RC) milestone Apr 1, 2016
@jmarolf jmarolf modified the milestones: 2.0 (RC), 2.0 (Preview 5) Sep 23, 2016
@Pilchie Pilchie modified the milestones: 2.0 (RTM), 2.0 (RC) Nov 2, 2016
@Pilchie Pilchie modified the milestones: 3.0, 2.0 (RTM) Jan 19, 2017
@jmarolf jmarolf removed their assignment Mar 16, 2017
@jinujoseph jinujoseph modified the milestones: 16.0, 16.1 Jan 16, 2019
@jinujoseph jinujoseph modified the milestones: 16.1, Backlog Apr 24, 2019
@CyrusNajmabadi
Copy link
Member

Would take a small PR here. Remove unused usings should not touch anything but usings and only whitespace directly around them.

@CyrusNajmabadi CyrusNajmabadi added the help wanted The issue is "up for grabs" - add a comment if you are interested in working on it label Oct 20, 2024
@github-project-automation github-project-automation bot moved this to InQueue in Small Fixes Oct 20, 2024
@CyrusNajmabadi
Copy link
Member

Fixed with #76038.

@github-project-automation github-project-automation bot moved this from InQueue to Completed in Small Fixes Nov 29, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Area-IDE Bug help wanted The issue is "up for grabs" - add a comment if you are interested in working on it
Projects
Status: Completed
Development

No branches or pull requests

5 participants