search

consolidation / site-process

A thin wrapper around the Symfony Process Component that allows applications to use the Site Alias library to specify the target for a remote call.
Other
50 stars 19 forks source link

Allow to explicitly specify the transport to use for a site-alias #71

Open ceesgeene opened 1 year ago

ceesgeene commented 1 year ago

Overview

This pull request:

Q A
Bug fix? no
New feature? yes
Has tests? yes
BC breaks? no
Deprecations? no

Summary

This allows to explicitly specify the transport to use.

Description

This for example allows for the use of the host property, without automatically forcing the ssh transport.

codecov[bot] commented 1 year ago

Codecov Report

Merging #71 (1e51630) into main (49ec2b9) will increase coverage by 0.83%. The diff coverage is 91.66%.

@@             Coverage Diff              @@
##               main      #71      +/-   ##
============================================
+ Coverage     77.85%   78.68%   +0.83%     
- Complexity      174      177       +3     
============================================
  Files            20       20              
  Lines           438      441       +3     
============================================
+ Hits            341      347       +6     
+ Misses           97       94       -3
Impacted Files Coverage Δ
src/ProcessManager.php 76.74% <91.66%> (-0.76%) :arrow_down:
src/Transport/KubectlTransport.php 88.88% <0.00%> (+7.40%) :arrow_up:
src/Factory/KubectlTransportFactory.php 100.00% <0.00%> (+50.00%) :arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more