Skip to content
This repository was archived by the owner on Apr 15, 2024. It is now read-only.

Release0.12.14 c#182

Merged
rsrivastava-cs merged 17 commits intomasterfrom
release0.12.14-c
Apr 22, 2019
Merged

Release0.12.14 c#182
rsrivastava-cs merged 17 commits intomasterfrom
release0.12.14-c

Conversation

@vadan-cs
Copy link
Copy Markdown
Contributor

Ok I think this is a better PR. It's develop rebased on master, but without squashing so we don't loose the commit history. So release0.12.14-c == develop-rebased-master .

We can force set the develop to this hash and go from there.
Here's the previous PR for reference #181

jgielstra-cs and others added 17 commits April 18, 2019 17:02
CCF-9252 update generator for linux
#162)

* CCF-9286: return instanceId in the message when promoting a snapshot to an environment that was being returned by the api response but not getting included in the cli response

* updated message with review comments
…nstanceId is under `response.message.instanceId` (#164)
* working recursive upload of directory

* 0.12.12
* use sequential chunks of files to upload

* 0.12.13
* add protocol prefix if one is missing

* 0.12.14
Copy link
Copy Markdown
Contributor

@daldridge-cs daldridge-cs left a comment

Choose a reason for hiding this comment

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

LGTM

@ghost ghost requested a review from AgrimAsthana-CS April 22, 2019 14:13
Copy link
Copy Markdown
Contributor

@AgrimAsthana-CS AgrimAsthana-CS left a comment

Choose a reason for hiding this comment

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

Not critical but by convention.. PULL_REQUEST_TEMPLATE.md is normally placed inside a .github folder at the root of a git repo.. looks good to merge otherwise.

@rsrivastava-cs rsrivastava-cs merged commit cae8557 into master Apr 22, 2019
@jgielstra-cs jgielstra-cs deleted the release0.12.14-c branch June 11, 2019 19:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.