Skip to content
This repository was archived by the owner on Feb 13, 2019. It is now read-only.

Conversation

peterblazejewicz
Copy link
Member

This commit updates WebAPI project template to one coming with beta5:

  • new hosting.ini file added to project
  • updated project.json
  • new README.md based on current html README document from ASP.NET
  • the ValuesController renamed and moved into more proper directory structure
  • updated copy syntax to use fs-mem module
  • the tests are updated to new content and updated structure

This commit also updates WebAPI controller subgenerator as directly related to
WebAPI project and using the same content

@peterblazejewicz
Copy link
Member Author

previous PR:
#199

@sayedihashimi
Copy link
Member

@peterblazejewicz looks like there was a build failure related to Startup.cs

@peterblazejewicz
Copy link
Member Author

The commit is updated with @rustd comments and rebased and squashed with current beta5.

It does restore OK now, but fails on build and run:
https://gist.github.com/peterblazejewicz/899c5b595792a739e264
https://gist.github.com/peterblazejewicz/41015b681d27817d6543
https://gist.github.com/peterblazejewicz/ef70a7ebce5e965e6efc

I can see a lot of beta4 bits mixed with beta5 in this setups.

@peterblazejewicz
Copy link
Member Author

The Travis is failing on tests that pass locally:

  aspnet - Web API Application
    Checking directories
      ✓ Application directory created
      ✓ Controllers directory created
      ✓ Properties directory created
      ✓ wwwroot directory created
    Checking files
      ✓ webAPITest/Controllers/ValuesController.cs created.
      ✓ webAPITest/hosting.ini created.
      ✓ webAPITest/project.json created.
      ✓ webAPITest/Properties/launchSettings.json created.
      ✓ webAPITest/Startup.cs created.
      ✓ webAPITest/wwwroot/README.md created.

This commit updates WebAPI project template to one coming with beta5:
- new hosting.ini file added to project
- updated project.json
- new README.md based on current html README document from ASP.NET
- the ValuesController renamed and moved into more proper directory structure
- the tests are updated to new content and updated structure
This commit also updates WebAPI controller subgenerator as directly related to
WebAPI project and using the same content
@sayedihashimi sayedihashimi merged commit 8956874 into OmniSharp:beta5 Jun 21, 2015
@sayedihashimi
Copy link
Member

Thanks @peterblazejewicz I've merged it. I think the test was failing because of a casing issue, see e6681f2.

I was able to build and run w/o errors. If you are still having errors try the following:

  • delete packages folder in ~/.dnx/packages
  • set dnvm to beta5
  • restore packages with dnu restore -f https://nuget.org/api/v2
  • build and run

Note: I am still unable to stop kestrel on Mac such that the port is released. The only way I found to close kestrel is with CTRL+Z and that keeps the port in use. See my previous comment #191 (comment)

@peterblazejewicz peterblazejewicz deleted the update/webapi-project branch June 21, 2015 09:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants