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

Update the solution to using .NET 4.5 #634

Closed
wants to merge 2 commits into from
Closed

Update the solution to using .NET 4.5 #634

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Nov 8, 2012

This also includes updating the build Azure package scripts. The Windows Azure SDK 1.8 is required to build the Azure package with this change.

public static UsersController Users = new T4MVC_UsersController();
public static SharedController Shared = new SharedController();
[GeneratedCode("T4MVC", "2.0"), DebuggerNonUserCode]
public static class MVC {
Copy link
Member

Choose a reason for hiding this comment

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

It looks like the brace style changed in this file and it no longer matches our standard. Is that controllable?

Copy link
Author

Choose a reason for hiding this comment

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

Just talked to David Ebbo. It's not controllable with the current version of T4MVC. He said he'll fix it in the next version. :)

Copy link
Member

Choose a reason for hiding this comment

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

Okay, no problem. If it isn't controllable, then it shouldn't change again, and that's fine.

@jeffhandley
Copy link
Member

Can you pull this into Preview instead of master please?

@ghost
Copy link
Author

ghost commented Nov 9, 2012

OK, close this pull request.

@ghost ghost closed this Nov 9, 2012
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant