Skip to content

Conversation

jonwis
Copy link
Member

@jonwis jonwis commented May 7, 2020

Bring up initial markdown documentation, solution, "common" type, and get things building.

Much more work to be done:

  • Add more interesting code
  • Add the docs/ content promised in the README
  • Tests
  • Samples
  • Generate a Framework Package
  • Generate a NuGet package with metadata & Framework Package reference
  • Get build tools working

@@ -0,0 +1,269 @@
## Ignore Visual Studio temporary files, build results, and
Copy link
Contributor

Choose a reason for hiding this comment

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

Not a fan of the kitchen sink .gitignore that Visual Studio generates. I'd rather be deliberate about what I ignore.

Copy link
Member

Choose a reason for hiding this comment

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

Most of this is probably unneeded if you also pick up the directory.props and other root configurations that make sure all the VS generated goop is in the root "BuildOutput" folder. VS generates quite a lot of nonsense into the source tree if you aren't careful though and I feel this is safer than folks accidentally blowing up the remote git repo with binary blobs that are never meant to be checked in.

{A4412A57-CECE-4494-A873-CCBBA5C6E9E6}.Debug|x86.ActiveCfg = Debug|Win32
{A4412A57-CECE-4494-A873-CCBBA5C6E9E6}.Debug|x86.Build.0 = Debug|Win32
{A4412A57-CECE-4494-A873-CCBBA5C6E9E6}.Release|ARM.ActiveCfg = Release|ARM
{A4412A57-CECE-4494-A873-CCBBA5C6E9E6}.Release|ARM.Build.0 = Release|ARM
Copy link
Member

Choose a reason for hiding this comment

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

We should make sure we have ARM64 configs from the get-go.

@jonwis jonwis closed this May 15, 2020
@jonwis jonwis deleted the user/jonwis/initial-content branch May 15, 2020 04:03
@weitzhandler weitzhandler mentioned this pull request May 21, 2020
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.

3 participants