Skip to content

Improve out-of-box experience (OOBE) for new user's first build#48

Open
jspraul wants to merge 5 commits intosciserver:v1.2/developfrom
jspraul:v1.2/develop
Open

Improve out-of-box experience (OOBE) for new user's first build#48
jspraul wants to merge 5 commits intosciserver:v1.2/developfrom
jspraul:v1.2/develop

Conversation

@jspraul
Copy link

@jspraul jspraul commented Nov 11, 2016

Some minor tweaks and clean-up to improve a new user's first build.

  • Reduce .NET framework to v4.5 in all projects referenced in solutions in source control
  • Fix ProjectGuid conflicts (entities solution disagreeing with everything else)
    • Also include missing Components project in entities solution
  • Fix NuGet hint paths so that projects build cleanly
  • Add a solution to build the SqlParser UI
  • Change SqlParser UI to stop processing if parsing fails (avoid NullReferenceException)

I tried to keep the changes minimal and granular enough that you can pick and choose what you want to apply, if anything. Unfortunately the .NET 4.5 change (to reach a slightly wider audience) was done before the hint path change, so there may be a slight conflict if you want the hint path fixed without .NET 4.5.

There is a lot of interest in parsing SQL in C#; you might gain some developers if you document more about how you're doing that!

Any documentation on how to build (still have to build entities solution twice!) and use the project would be a great addition to the README. Thanks for your consideration & keep up the good work!

@dobos
Copy link
Member

dobos commented Nov 14, 2016

Hey,

I'm glad you found it useful. I'm currently working on building issues and continuous integration but this is a big and complex project and I'm the only developer right now. I was also considering factoring out the parser project entirely, maybe will do it sometime. I'll take a look at your pull request sometime this week.

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