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

webapps: Updating default node version used during creates #7821

Merged
merged 3 commits into from
Nov 13, 2018

Conversation

panchagnula
Copy link
Contributor


Fixes: #7749, #7689
This checklist is used to make sure that common guidelines for a pull request are followed.

  • The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

  • I adhere to the Command Guidelines.

@panchagnula
Copy link
Contributor Author

Need to update history will update with the next iteration

@panchagnula panchagnula force-pushed the sisirap-CLI-bugFixes-dev branch from d464de7 to f3254d0 Compare November 13, 2018 08:44
@panchagnula panchagnula added the App Services az appservice label Nov 13, 2018
Copy link
Contributor

@yugangw-msft yugangw-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor comment. The rest LGTM.

return poller
elif action == 'preview':
if target_slot is None:
result = client.web_apps.apply_slot_config_to_production(resource_group_name,
webapp, slot, True)
else:
result = client.web_apps.apply_slot_configuration_slot(resource_group_name, webapp,
slot, target_slot, True)
slot, (target_slot or 'production'), True)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we know target_slot is not None, so if should just be target_slot

@yugangw-msft yugangw-msft merged commit b4e6d76 into Azure:dev Nov 13, 2018
@panchagnula panchagnula deleted the sisirap-CLI-bugFixes-dev branch April 6, 2019 01:57
@haroldrandom haroldrandom added the App Services az appservice label Oct 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App Services az appservice
Projects
None yet
Development

Successfully merging this pull request may close these issues.

webapp:Node version should be updated for functionapp create
3 participants