-
Notifications
You must be signed in to change notification settings - Fork 0
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
Backup to google storage #57
Conversation
ccc4dbc
to
d3f61b6
Compare
6dffcff
to
3aecad7
Compare
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.
Ještě doplnit do README, jinak LGTM 👍
self::assertGreaterThan(0, count($events)); | ||
} | ||
|
||
public function testCreateUnexistsBackupFolder(): void |
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.
nonexistent
místo unexists
asi
díky, README jsem doplnil |
case Config::STORAGE_BACKEND_GCS: | ||
foreach (['#jsonKey', '#bucket', 'region'] as $item) { | ||
if (empty($v[$item])) { | ||
throw new InvalidConfigurationException(sprintf( | ||
'Missing required parameter "%s".', | ||
$item, | ||
)); | ||
} | ||
} | ||
break; |
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.
Testy maji jeste rezervy
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.
doplnil jsem je 896748c
https://keboola.atlassian.net/browse/PST-2374