Skip to content
This repository was archived by the owner on May 28, 2021. It is now read-only.

fix backups to disk #209

Merged
merged 1 commit into from
Aug 24, 2018
Merged

fix backups to disk #209

merged 1 commit into from
Aug 24, 2018

Conversation

Simon-Li
Copy link

@Simon-Li Simon-Li commented Aug 19, 2018

this is for #147

@prydie prydie added needs-approval oracle-cla: no Contributor has not yet signed the Oracle Contributor Licence Agreement and removed needs-approval labels Aug 20, 2018
@prydie
Copy link

prydie commented Aug 20, 2018

Awesome, thanks @Simon-Li! Changes LGTM 👍

Would you mind rebasing your PR into a singe signed-off commit? Additionally, if you haven't already signed to Oracle Contributor Agreement can you please do so.

@owainlewis Can we get the CI hooked up for third-party PRs and trigger a run for this prior to merging?

Signed-off-by: SimonLi <simon.jiyou@gmail.com>
@Simon-Li Simon-Li force-pushed the fix-backups-to-disk branch from 0a17264 to 8e97239 Compare August 20, 2018 14:33
@Simon-Li
Copy link
Author

Rebased and the OCA was submitted, thanks @prydie

@owainlewis
Copy link
Member

owainlewis commented Aug 23, 2018

Thanks @Simon-Li we are just waiting for an approval on OCA and then we'll get this merged.

@prydie
Copy link

prydie commented Aug 23, 2018

@Simon-Li Did you submit the OCA from a different email address perchance? The approvals team can't seem to find your submission I'm afraid.

I spoke to soon - they've got it!

@prydie prydie added oracle-cla: yes Contributor has signed the Oracle Contributor Licence Agreement and removed oracle-cla: no Contributor has not yet signed the Oracle Contributor Licence Agreement labels Aug 24, 2018
@prydie prydie merged commit ee732cf into oracle:master Aug 24, 2018
@prydie
Copy link

prydie commented Aug 24, 2018

Thanks for your contribution @Simon-Li!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement feature oracle-cla: yes Contributor has signed the Oracle Contributor Licence Agreement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants