-
Notifications
You must be signed in to change notification settings - Fork 455
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
Azure Functions Cli #477
Azure Functions Cli #477
Conversation
Hi @ahmelsayed, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution! The agreement was validated by .NET Foundation and real humans are currently evaluating your PR. TTYL, DNFBOT; |
Hi @ahmelsayed, as a .Net developer doing Node development, this is the interface I would be most comfortable with. Especially since I use a Mac and Atom for my development environment. The command are logical, well thought out and not surprising. That's always a plus. |
@@ -0,0 +1,42 @@ | |||
<?xml version="1.0" encoding="utf-8"?> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you name this project (and related projects) WebJobs.Script.CLI? That leads to less confusion with the existing 2 hosts we have, which are about hosting functions, not CLI stuff.
Also you'll want to start a wiki page on how to use the CLI :) At least getting people started. |
{ | ||
public class FileTracer : TraceWriter | ||
{ | ||
private Collection<TraceEvent> _traces = new Collection<TraceEvent>(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unused?
can you try |
Still having issues running this on my machine. I'm getting an error thrown from Ln23 of ConsoleApp.cs. This repros on 0.0.9 as well.
Looks like Storage issues? I tried setting environment variables/etc. I am also still getting install issues with 0.0.10, but I manually grabbed the tarball, unzipped it, and placed it in my global npm. It runs fine, except for
It's quite hard to tell if I'm doing something wrong or if it's just not functional. Attaching a debugger only kept me going through a loop where I was getting the above exceptions being thrown. Ideas for next steps? |
are you running |
Also the web command is the only command that actually loads the host, so it's the one most likely to fail because of a missing dependency. |
I'm not running func.exe/func.vshost.exe directly in either case. In one, I attached the debugger to the source in your fork to find where the exception was being thrown. In the other, it was just the npm package that gets installed from npm registry. I tried attaching to the host to see if I can find in more detail what was causing the host to not start, but symbols weren't loading for the script.host |
Also, I realized I didn't give verbose logs for the npm install issue:
It doesn't look like it's happening in your code, but I've not having this happen on any other package I've tried lately. |
I found the issue with run/web not working - I had a environment variable on all of my machines from a previous experiment which was being picked up and was broken. :) Works nicely now. Having a blast! |
That probably does highlight some of the issues with our current model for running the host in background - we need a better debugging experience for host issues. |
> npm i -g azurefunctions
Local -> Azure:
Azure -> Local: