Skip to content

Commit

Permalink
Merge branch 'master' into malhusseiny
Browse files Browse the repository at this point in the history
  • Loading branch information
malhusseiny authored Jan 26, 2021
2 parents 53c80dc + 9e21c49 commit 72f4ebf
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 13 deletions.
5 changes: 4 additions & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,9 +33,12 @@ jobs:
with:
dependency-versions: 'locked'

- name: Static Code Analysis
- name: Code style
run: bin/phpcs -s -n --standard=./ruleset.xml classes src tests/classes tests/Phing

- name: Static analysis (level 1)
run: bin/phpstan

test:
needs:
- pre_job
Expand Down
11 changes: 0 additions & 11 deletions classes/phing/tasks/ext/Service/Amazon/S3.php
Original file line number Diff line number Diff line change
Expand Up @@ -112,17 +112,6 @@ public function getObjectInstance($object)
return $this->getClientInstance()->getObject($object);
}

/**
* Check if the object already exists in the current bucket
*
* @param mixed $object
* @return bool
*/
public function isObjectAvailable($object)
{
return (bool) $this->getObjectInstance($object)->load(Services_Amazon_S3_Resource_Object::LOAD_METADATA_ONLY);
}

/**
* Returns an instance of Services_Amazon_S3_Resource_Bucket
*
Expand Down
2 changes: 2 additions & 0 deletions classes/phing/tasks/ext/VersionTask.php
Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,8 @@ public function setPropFile($isPropFile)
*/
public function main()
{
$properties = null;

// check supplied attributes
$this->checkReleasetype();
$this->checkFile();
Expand Down
2 changes: 1 addition & 1 deletion classes/phing/tasks/ext/visualizer/VisualizerTask.php
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ class VisualizerTask extends HttpTask
public function init(): void
{
parent::init();
if (!function_exists(\Jawira\PlantUml\encodep::class)) {
if (!function_exists('\Jawira\PlantUml\encodep')) {
$exceptionMessage = get_class($this) . ' requires "jawira/plantuml-encoding" library';
}
if (!class_exists(XSLTProcessor::class)) {
Expand Down
1 change: 1 addition & 0 deletions phpstan.neon
Original file line number Diff line number Diff line change
Expand Up @@ -7,3 +7,4 @@ parameters:
- tests/Phing
bootstrapFiles:
- tests/bootstrap.php
- vendor/pear/versioncontrol_svn/VersionControl/SVN.php

0 comments on commit 72f4ebf

Please sign in to comment.