Skip to content

Test v2.x branch #259

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

Merged
merged 1 commit into from
Nov 21, 2019
Merged

Test v2.x branch #259

merged 1 commit into from
Nov 21, 2019

Conversation

mhoeger
Copy link
Contributor

@mhoeger mhoeger commented Nov 19, 2019

This change adds v2,x as a branch to test. v2.x branch currently (see changes here):

TODO:

  • remove Node.js v10 support?
    • potentially leverage Node.js v12 features
  • [breaking] fix camelCase of timer trigger object: camelCase Timer Trigger object #188
    • potentially add to .proto
  • [breaking] consider removing context.req and context.res objects: Remove duplication / improve organization on "context" properties #204
  • [breaking] in host and worker, make sure http payload "as raw" as possible
  • explore options for bugs with protobuf.js and windows (Issue 1303, Issue 1275)
    • protoc or just default Linux and accept Windows bugs? Probably document windows bugs

@mhoeger
Copy link
Contributor Author

mhoeger commented Nov 19, 2019

@ColbyTresness and @pragnagopa and @fabiocav

@ColbyTresness
Copy link

We decided with @fabiocav not to remove Node 10 here. Other than that looks good

@mhoeger mhoeger merged commit ca042e6 into Azure:v2.x Nov 21, 2019
mhoeger added a commit to mhoeger/azure-functions-nodejs-worker that referenced this pull request Jan 14, 2020
mhoeger added a commit that referenced this pull request Jan 15, 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.

2 participants