Skip to content

Commit

Permalink
Updated dependencies to the latest versions and fixed coding standard…
Browse files Browse the repository at this point in the history
…s. (#35)
  • Loading branch information
Alex Skrypnyk authored May 1, 2022
1 parent 503d256 commit bf1ca4b
Show file tree
Hide file tree
Showing 19 changed files with 327 additions and 220 deletions.
2 changes: 1 addition & 1 deletion .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,11 +30,11 @@ jobs:
steps:
- attach_workspace:
at: /workspace
- checkout
- add_ssh_keys:
fingerprints:
- *deploy_ssh_fingerprint
- *step_setup_git
- checkout
- run: composer validate --ansi --strict
- run: composer install
- run: composer lint
Expand Down
20 changes: 20 additions & 0 deletions .editorconfig
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
# Drupal editor configuration normalization
# @see http://editorconfig.org/

# This is the top-most .editorconfig file; do not search in parent directories.
root = true

# All files.
[*]
end_of_line = LF
indent_style = space
indent_size = 4
charset = utf-8
trim_trailing_whitespace = true
insert_final_newline = true

[composer.{json,lock}]
indent_size = 4

[*.xml]
indent_size = 4
1 change: 1 addition & 0 deletions .gitattributes
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

.gitattributes export-ignore
.gitignore export-ignore
.editorconfig export-ignore
tests export-ignore
.circle export-ignore
phpcs.xml export-ignore
Expand Down
1 change: 1 addition & 0 deletions .phpunit.result.cache
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
{"version":1,"defects":{"IntegratedExperts\\Robo\\Tests\\Integration\\BranchTest::testBuild":4,"IntegratedExperts\\Robo\\Tests\\Integration\\BranchTest::testBuildMoreCommitsSameBranch":4,"IntegratedExperts\\Robo\\Tests\\Integration\\BranchTest::testBuildMoreCommits":4,"IntegratedExperts\\Robo\\Tests\\Integration\\BranchTest::testCleanupAfterSuccess":4,"IntegratedExperts\\Robo\\Tests\\Integration\\BranchTest::testCleanupAfterFailure":4,"IntegratedExperts\\Robo\\Tests\\Integration\\BranchTest::testGitignore":4,"IntegratedExperts\\Robo\\Tests\\Integration\\BranchTest::testGitignoreCustom":4,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testBuild":4,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testBuildMoreCommits":4,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testIdempotence":4,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testSubRepos":4,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testCleanupAfterSuccess":4,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testCleanupAfterFailure":3,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testGitignore":4,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testGitignoreCustom":4,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testGitignoreCustomRemoveCommittedFiles":4,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testGitignoreCustomWhitelisting":4,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testBuildTag":4,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testBuildMultipleTags":4,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testBuildMultipleTagsDelimiter":4,"IntegratedExperts\\Robo\\Tests\\Integration\\GeneralTest::testInfo":4,"IntegratedExperts\\Robo\\Tests\\Integration\\GeneralTest::testShowChanges":4,"IntegratedExperts\\Robo\\Tests\\Integration\\GeneralTest::testNoCleanup":4,"IntegratedExperts\\Robo\\Tests\\Integration\\GeneralTest::testReport":4,"IntegratedExperts\\Robo\\Tests\\Integration\\GeneralTest::testDebug":4,"IntegratedExperts\\Robo\\Tests\\Integration\\GeneralTest::testDebugDisabled":4,"IntegratedExperts\\Robo\\Tests\\Integration\\TagTest::testDetachedTag":4},"times":{"IntegratedExperts\\Robo\\Tests\\Integration\\BranchTest::testBuild":1.482,"IntegratedExperts\\Robo\\Tests\\Integration\\BranchTest::testBuildMoreCommitsSameBranch":3.087,"IntegratedExperts\\Robo\\Tests\\Integration\\BranchTest::testBuildMoreCommits":3.061,"IntegratedExperts\\Robo\\Tests\\Integration\\BranchTest::testCleanupAfterSuccess":1.62,"IntegratedExperts\\Robo\\Tests\\Integration\\BranchTest::testCleanupAfterFailure":3.13,"IntegratedExperts\\Robo\\Tests\\Integration\\BranchTest::testGitignore":2.85,"IntegratedExperts\\Robo\\Tests\\Integration\\BranchTest::testGitignoreCustom":1.067,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testBuild":1.394,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testBuildMoreCommits":3.21,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testIdempotence":2.926,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testSubRepos":0.552,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testCleanupAfterSuccess":1.419,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testCleanupAfterFailure":0.808,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testGitignore":2.782,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testGitignoreCustom":1.021,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testGitignoreCustomRemoveCommittedFiles":0.429,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testGitignoreCustomWhitelisting":0.545,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testBuildTag":1.397,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testBuildMultipleTags":1.754,"IntegratedExperts\\Robo\\Tests\\Integration\\ForcePushTest::testBuildMultipleTagsDelimiter":1.7,"IntegratedExperts\\Robo\\Tests\\Integration\\GeneralTest::testPresence":0.376,"IntegratedExperts\\Robo\\Tests\\Integration\\GeneralTest::testHelp":0.285,"IntegratedExperts\\Robo\\Tests\\Integration\\GeneralTest::testCompulsoryParameter":0.319,"IntegratedExperts\\Robo\\Tests\\Integration\\GeneralTest::testInfo":1.083,"IntegratedExperts\\Robo\\Tests\\Integration\\GeneralTest::testShowChanges":0.888,"IntegratedExperts\\Robo\\Tests\\Integration\\GeneralTest::testNoCleanup":0.897,"IntegratedExperts\\Robo\\Tests\\Integration\\GeneralTest::testReport":0.891,"IntegratedExperts\\Robo\\Tests\\Integration\\GeneralTest::testDebug":0.996,"IntegratedExperts\\Robo\\Tests\\Integration\\GeneralTest::testDebugDisabled":1.009,"IntegratedExperts\\Robo\\Tests\\Integration\\TagTest::testDetachedTag":1.765,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testTokenProcess with data set #0":0.098,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testTokenProcess with data set #1":0.134,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testTokenProcess with data set #2":0.125,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testTokenProcess with data set #3":0.173,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testTokenProcess with data set #4":0.154,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testTokenProcess with data set #5":0.099,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testTokenProcess with data set #6":0.121,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testTokenProcess with data set #7":0.112,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testTokenProcess with data set #8":0.114,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testTokenProcess with data set #9":0.118,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testHasToken with data set #0":0.117,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testHasToken with data set #1":0.121,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testHasToken with data set #2":0.117,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testHasToken with data set #3":0.234,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testHasToken with data set #4":0.118,"IntegratedExperts\\Robo\\Tests\\Integration\\TokenTest::testHasToken with data set #5":0.102,"IntegratedExperts\\Robo\\Tests\\Unit\\ExcludeTest::testExcludeExists":0.112,"IntegratedExperts\\Robo\\Tests\\Unit\\ExcludeTest::testExcludeEmpty with data set #0":0.107,"IntegratedExperts\\Robo\\Tests\\Unit\\ExcludeTest::testExcludeEmpty with data set #1":0.102,"IntegratedExperts\\Robo\\Tests\\Unit\\ExcludeTest::testExcludeEmpty with data set #2":0.116,"IntegratedExperts\\Robo\\Tests\\Unit\\ExcludeTest::testExcludeEmpty with data set #3":0.114,"IntegratedExperts\\Robo\\Tests\\Unit\\ExcludeTest::testExcludeEmpty with data set #4":0.132,"IntegratedExperts\\Robo\\Tests\\Unit\\ExcludeTest::testExcludeEmpty with data set #5":0.14,"IntegratedExperts\\Robo\\Tests\\Unit\\ExcludeTest::testExcludeEmpty with data set #6":0.113,"IntegratedExperts\\Robo\\Tests\\Unit\\ExcludeTest::testExcludeEmpty with data set #7":0.121,"IntegratedExperts\\Robo\\Tests\\Unit\\ExcludeTest::testExcludeEmpty with data set #8":0.113,"IntegratedExperts\\Robo\\Tests\\Unit\\ExcludeTest::testExcludeEmpty with data set #9":0.118,"IntegratedExperts\\Robo\\Tests\\Unit\\ExcludeTest::testExcludeEmpty with data set #10":0.109,"IntegratedExperts\\Robo\\Tests\\Unit\\ExcludeTest::testExcludeEmpty with data set #11":0.114,"IntegratedExperts\\Robo\\Tests\\Unit\\ExcludeTest::testExcludeEmpty with data set #12":0.114,"IntegratedExperts\\Robo\\Tests\\Unit\\ExcludeTest::testExcludeEmpty with data set #13":0.111}}
75 changes: 41 additions & 34 deletions composer.json
Original file line number Diff line number Diff line change
@@ -1,37 +1,44 @@
{
"name": "integratedexperts/robo-git-artefact",
"description": "Robo task to push git artefact to remote repository",
"type": "robo-tasks",
"license": "GPL-2.0-or-later",
"authors": [
{
"name": "Alex Skrypnyk",
"email": "[email protected]"
"name": "integratedexperts/robo-git-artefact",
"description": "Robo task to push git artefact to remote repository",
"type": "robo-tasks",
"license": "GPL-2.0-or-later",
"authors": [
{
"name": "Alex Skrypnyk",
"email": "[email protected]"
}
],
"require": {
"consolidation/robo": "^3"
},
"require-dev": {
"squizlabs/php_codesniffer": "^3.6",
"phpunit/phpunit": "^9.5",
"dealerdirect/phpcodesniffer-composer-installer": "^0.7",
"escapestudios/symfony2-coding-standard": "^3"
},
"scripts": {
"lint": "./vendor/bin/phpcs -s",
"test": "./vendor/bin/phpunit --colors=always --fail-on-warning",
"clean": "vagrant -f destroy &>/dev/null || $(which true); rm -Rf $(pwd)/.vagrant; rm -Rf $(pwd)/.beetbox; rm -Rf $(pwd)/Vagrantfile; rm -Rf $(pwd)/vendor; rm -Rf $(pwd)/screenshots;",
"install-codestandards": [
"Dealerdirect\\Composer\\Plugin\\Installers\\PHPCodeSniffer\\Plugin::run"
]
},
"autoload": {
"psr-4": {
"IntegratedExperts\\Robo\\": "src"
}
},
"autoload-dev": {
"psr-4": {
"IntegratedExperts\\Robo\\Tests\\": "tests"
}
},
"config": {
"allow-plugins": {
"dealerdirect/phpcodesniffer-composer-installer": true
}
}
],
"require": {
"php": ">=5.5.0",
"consolidation/robo": "~1"
},
"require-dev": {
"squizlabs/php_codesniffer": "2.*",
"escapestudios/symfony2-coding-standard": "~2.0",
"phpunit/phpunit": "^6.3",
"dealerdirect/phpcodesniffer-composer-installer": "^0.5.0"
},
"scripts": {
"lint": "./vendor/bin/phpcs",
"test": "./vendor/bin/phpunit --colors=always --fail-on-warning",
"clean": "vagrant -f destroy &>/dev/null || $(which true); rm -Rf $(pwd)/.vagrant; rm -Rf $(pwd)/.beetbox; rm -Rf $(pwd)/Vagrantfile; rm -Rf $(pwd)/vendor; rm -Rf $(pwd)/screenshots;"
},
"autoload": {
"psr-4": {
"IntegratedExperts\\Robo\\": "src"
}
},
"autoload-dev": {
"psr-4": {
"IntegratedExperts\\Robo\\Tests\\": "tests"
}
}
}
16 changes: 13 additions & 3 deletions phpcs.xml
Original file line number Diff line number Diff line change
Expand Up @@ -2,21 +2,31 @@
<ruleset name="robo-git-artefact">
<description>The coding standard for behat-relativity.</description>

<rule ref="Symfony2"/>
<rule ref="Symfony"/>

<file>src</file>
<file>tests</file>

<arg name="colors"/>
<arg value="sp"/>

<rule ref="Symfony.Commenting.License.Warning">
<exclude-pattern>*.*</exclude-pattern>
</rule>
<rule ref="Symfony.Functions.Arguments.Invalid">
<exclude-pattern>*.*</exclude-pattern>
</rule>
<rule ref="Symfony.ControlStructure.YodaConditions.Invalid">
<exclude-pattern>*.*</exclude-pattern>
</rule>

<!--Allow uncommented functions in tests as they usually provide enough
information from their names.-->
<rule ref="Symfony2.Commenting.FunctionComment.Missing">
<rule ref="Symfony.Commenting.FunctionComment.Missing">
<exclude-pattern>tests/*.Test\.php</exclude-pattern>
<exclude-pattern>tests/*.TestCase\.php</exclude-pattern>
</rule>
<rule ref="Symfony2.Commenting.FunctionComment.MissingParamTag">
<rule ref="Symfony.Commenting.FunctionComment.MissingParamTag">
<exclude-pattern>tests/*.Test\.php</exclude-pattern>
<exclude-pattern>tests/*.TestCase\.php</exclude-pattern>
</rule>
Expand Down
6 changes: 0 additions & 6 deletions phpunit.xml
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,4 @@
<directory>./tests/</directory>
</testsuite>
</testsuites>
<!-- Filter for coverage reports. -->
<filter>
<blacklist>
<directory>./vendor</directory>
</blacklist>
</filter>
</phpunit>
Loading

0 comments on commit bf1ca4b

Please sign in to comment.