Skip to content

Add template with code in a Main method #647

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

Merged
merged 1 commit into from
Oct 26, 2022

Conversation

jskeet
Copy link
Contributor

@jskeet jskeet commented Oct 26, 2022

This will allow for examples which are just statements. (Also adds example in 16.1 Arrays to show how to use it.)

Towards #646

This will allow for examples which are just statements.
(Also adds example in 16.1 Arrays to show how to use it.)

Towards dotnet#646
@jskeet jskeet requested a review from RexJaeschke October 26, 2022 13:07
@RexJaeschke
Copy link
Contributor

@jskeet, I'm confused! This isn't one of the scenarios I reported in the new issue. In fact, I was able to get that example (and other examples like it) to build by using the template standalone-console, and taking advantage of the ability to have statements at the top level in recent compiler versions.

@jskeet
Copy link
Contributor Author

jskeet commented Oct 26, 2022

It may not be necessary if we're happy to use a later compiler to test C# 7, but it still feels like it's part of fixing "incomplete example". (I don't have the full context of what doesn't work as we can't see your code with the metadata, unless you've pushed that to a branch I'm not aware of. So I'm going on descriptions basically.)

Copy link
Contributor

@RexJaeschke RexJaeschke left a comment

Choose a reason for hiding this comment

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

Yes, it sounds like a good idea to have this new template.

@RexJaeschke RexJaeschke merged commit d24bf4a into dotnet:draft-v7 Oct 26, 2022
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.

2 participants