deployphp / deployer

tests/legacy/recipe/parallel.php

<?php

namespace Deployer;

localhost('prod');
localhost('beta')
    ->set('host_level_callback_config', function () {

The method set() does not exist on Deployer\Support\ObjectProxy. Since you implemented __call, consider adding a @method annotation.

        return 'from callback';
    });

// testServer:

task('ask', function () {
    $answer = ask('Question: What kind of bear is best?');
    writeln($answer);

It seems like $answer can also be of type null; however, parameter $message of Deployer\writeln() does only seem to accept string, maybe add an additional type check?

});

// testWorker, testOption:

set('greet', '_');

task('echo', function () {
    $alias = currentHost()->getAlias();
    run("echo {{greet}}, $alias!");
});

// testCachedHostConfig:

set('upper_host', function () {
    writeln('running ' . (Deployer::isWorker() ? 'worker' : 'master') . ' on ' . currentHost()->getAlias());
    return strtoupper(currentHost()->getAlias());

It seems like currentHost()->getAlias() can also be of type null; however, parameter $string of strtoupper() does only seem to accept string, maybe add an additional type check?

});

task('cache_config_test', function () {
    writeln('echo 1: {{upper_host}}');
});

after('cache_config_test', function () {
    writeln('echo 2: {{upper_host}}');
});

// testHostConfigFromCallback:

set('host_level_callback_config', 'from global');

task('host_config_from_callback', function () {
    writeln('config value is {{host_level_callback_config}}');
});