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

File headers not handled correctly when usingDirectivesPlacement is outsideNamespace #1941

Closed
sharwell opened this issue Dec 10, 2015 · 3 comments
Assignees
Milestone

Comments

@sharwell
Copy link
Member

Code to reproduce:

// Copyright (c) Tunnel Vision Laboratories, LLC. All Rights Reserved.
// Licensed under the Apache License, Version 2.0. See LICENSE in the project root for license information.

namespace NamespaceName
{
    using System;

    public class ClassName
    {
    }
}

Expected code fix result:

// Copyright (c) Tunnel Vision Laboratories, LLC. All Rights Reserved.
// Licensed under the Apache License, Version 2.0. See LICENSE in the project root for license information.

using System;

namespace NamespaceName
{
    public class ClassName
    {
    }
}

Actual code fix result:

// Copyright (c) Tunnel Vision Laboratories, LLC. All Rights Reserved.
// Licensed under the Apache License, Version 2.0. See LICENSE in the project root for license information.

using System;

// Copyright (c) Tunnel Vision Laboratories, LLC. All Rights Reserved.
// Licensed under the Apache License, Version 2.0. See LICENSE in the project root for license information.

namespace NamespaceName
{
    public class ClassName
    {
    }
}
@vweijsters
Copy link
Contributor

Grabbing this

@vweijsters
Copy link
Contributor

@sharwell I'm having a hard time reproducing your actual result. Are you sure that it is correct?

I'm seeing the following faulty result:

using System;

// Copyright (c) Tunnel Vision Laboratories, LLC. All Rights Reserved.
// Licensed under the Apache License, Version 2.0. See LICENSE in the project root for license information.

namespace NamespaceName
{
    public class ClassName
    {
    }
}

@sharwell
Copy link
Member Author

I'm running this in Update 1, but yes I just verified that the example behaves as described.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants