Skip to content

Commit 76fbe3b

Browse files
committed
tweaking release process
1 parent fd75ae5 commit 76fbe3b

File tree

6 files changed

+13
-89
lines changed

6 files changed

+13
-89
lines changed

.env.dist

Lines changed: 0 additions & 1 deletion
This file was deleted.

bin/create_release

Lines changed: 7 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -4,29 +4,15 @@
44
require __DIR__.'/../vendor/autoload.php';
55

66
use SymfonyDocsBuilder\Phar\Compiler;
7-
use SymfonyDocsBuilder\Release\Exception\ReleaseFailed;
87
use SymfonyDocsBuilder\Release\GithubApiHttpClientFactory;
98
use SymfonyDocsBuilder\Release\Releaser;
109

11-
error_reporting(-1);
12-
ini_set('display_errors', 1);
10+
if ($argc !== 4) {
11+
echo 'Not enough arguments. usage: "./bin/release tag [release_name [release_description]]"';
12+
echo "\n\n";
1313

14-
try {
15-
$releaser = new Releaser((new GithubApiHttpClientFactory())->createHttpClient(), new Compiler());
16-
17-
if ($argc === 1) {
18-
throw new RuntimeException('Not enough arguments. usage: "./bin/release tag [release_name [release_description]]"');
19-
}
20-
21-
array_shift($argv);
22-
23-
$releaser->createRelease(...$argv);
24-
} catch (Exception $e) {
25-
if ($e instanceof ReleaseFailed) {
26-
echo $e->toString();
27-
} else {
28-
echo 'Failed to create a new release: ['.get_class($e).'] '.$e->getMessage().' at '.$e->getFile().':'.$e->getLine()."\n";
29-
}
30-
31-
exit(1);
14+
return 1;
3215
}
16+
17+
$releaser = new Releaser((new GithubApiHttpClientFactory())->createHttpClient(), new Compiler());
18+
$releaser->createRelease($argv[1], $argv[2], $argv[3]);

composer.json

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@
2525
"require-dev": {
2626
"gajus/dindent": "^2.0",
2727
"symfony/phpunit-bridge": "^4.1",
28-
"symfony/process": "^4.2",
29-
"symfony/dotenv": "^4.3"
28+
"symfony/process": "^4.2"
3029
}
3130
}

composer.lock

Lines changed: 2 additions & 59 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/Release/GithubApiHttpClientFactory.php

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22

33
namespace SymfonyDocsBuilder\Release;
44

5-
use Symfony\Component\Dotenv\Dotenv;
65
use Symfony\Component\HttpClient\HttpClient;
76
use Symfony\Contracts\HttpClient\HttpClientInterface;
87

@@ -12,11 +11,8 @@ class GithubApiHttpClientFactory
1211

1312
public function __construct()
1413
{
15-
$dotenv = new Dotenv();
16-
$dotenv->load(__DIR__.'/../../.env');
17-
18-
if (empty($_SERVER['GITHUB_API_TOKEN'])) {
19-
throw new \RuntimeException('Please fill "GITHUB_API_TOKEN" in file "[PROJECT_DIR]/.env"');
14+
if (!isset($_SERVER['GITHUB_API_TOKEN'])) {
15+
throw new \RuntimeException('Missing "GITHUB_API_TOKEN" environment variable');
2016
}
2117

2218
$this->githubApiToken = $_SERVER['GITHUB_API_TOKEN'];

src/Release/Releaser.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
use Symfony\Contracts\HttpClient\Exception\HttpExceptionInterface;
66
use Symfony\Contracts\HttpClient\HttpClientInterface;
77
use SymfonyDocsBuilder\Phar\Compiler;
8+
use SymfonyDocsBuilder\Release\Exception\DeleteReleaseFailed;
89
use SymfonyDocsBuilder\Release\Exception\ReleaseFailed;
910

1011
class Releaser

0 commit comments

Comments
 (0)