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

Automatically do a build before deploying C# function projects #223

Closed
v-flyuan opened this issue Feb 13, 2018 · 2 comments
Closed

Automatically do a build before deploying C# function projects #223

v-flyuan opened this issue Feb 13, 2018 · 2 comments
Assignees
Labels
AT-CTI Created by CTI team enhancement P2
Milestone

Comments

@v-flyuan
Copy link

Repro steps:

  1. Create a new project using C#.
  2. Create a new function and add Httptrigger.
  3. Select 'Function'. Then the HttpTrigger file will be opened.
  4. Deploy to Azure Function App.
  5. Update the code.
  6. Deploy again. Verify whether you can get the updated content in Terminal window.

Expect:
The content is updated.

Actual:
The content isn't updated.

More Info:

  1. This issue doesn't reproduce on JavaScript.
  2. This issue also reproduce on Mac.
  3. If you update the code and F5. Then deploy again, you can get the updated content.
@ejizba ejizba changed the title The content isn't updated after deploying with updated code Automatically do a build before deploying C# function projects Feb 13, 2018
@ejizba
Copy link
Contributor

ejizba commented Feb 13, 2018

This is expected - you have to re-build your project before deploying. However, we could consider automatically doing that for the user - renamed issue to reflect that

@ejizba ejizba added the P2 label Feb 13, 2018
@ejizba ejizba added this to the future milestone Feb 13, 2018
@ejizba ejizba modified the milestones: future, 0.8.0 Mar 19, 2018
@ejizba
Copy link
Contributor

ejizba commented Mar 26, 2018

This requires waiting on the build task to complete before deploying, which isn't possible through the VS Code extension api today. However, it was just added to the proposed API here microsoft/vscode#45664

I think its worth waiting for that API rather than doing something hacky to enable this feature

cc @StephenWeatherford @nturinski

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
AT-CTI Created by CTI team enhancement P2
Projects
None yet
Development

No branches or pull requests

3 participants