Skip to content

Commit c25de67

Browse files
authored
[Feature] Add IsNumberEvaluator (#66)
* Add is number evaluator * Add containsNumber evaluator * up docs * fix lint * add test * add security check * add security check * fix make security command * up version * up version * up workflow * up test * fix test
1 parent 7edd54a commit c25de67

27 files changed

+724
-58
lines changed

.github/workflows/security.yml

+18
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
name: Security
2+
3+
on:
4+
push:
5+
branches: [ master ]
6+
pull_request:
7+
branches: [ master ]
8+
9+
jobs:
10+
security:
11+
name: Local PHP Security Checker
12+
runs-on: ubuntu-latest
13+
steps:
14+
- name: Checkout code
15+
uses: actions/checkout@v2
16+
17+
- name: Local PHP Security Checker
18+
uses: docker://pplotka/local-php-security-checker-github-actions

.github/workflows/testing.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@ name: Testing
22

33
on:
44
push:
5-
branches: [ master, 'dev' ]
5+
branches: [ master ]
66
pull_request:
7-
branches: [ master, 'dev' ]
7+
branches: [ master ]
88

99
jobs:
1010
testing:

CHANGELOG.md

+15-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,17 @@ This file contains changelogs.
44

55
[View all Releases](https://github.com/ArtARTs36/php-merge-request-linter/releases)
66

7-
## [Unreleased](https://github.com/ArtARTs36/php-merge-request-linter/compare/0.20.1..master)
7+
## [Unreleased](https://github.com/ArtARTs36/php-merge-request-linter/compare/0.20.2..master)
8+
9+
## [v0.20.2 (2023-09-13)](https://github.com/ArtARTs36/php-merge-request-linter/compare/0.20.1..0.20.2)
10+
11+
### Added
12+
* Added evaluator `isNumber`
13+
* Added evaluator `containsNumber`
14+
15+
[💾 Assets](https://github.com/ArtARTs36/php-merge-request-linter/releases/tag/0.20.2)
16+
17+
-----------------------------------------------------------------
818

919
## [v0.20.1 (2023-09-12)](https://github.com/ArtARTs36/php-merge-request-linter/compare/0.20.0..0.20.1)
1020

@@ -13,6 +23,8 @@ This file contains changelogs.
1323

1424
[💾 Assets](https://github.com/ArtARTs36/php-merge-request-linter/releases/tag/0.20.1)
1525

26+
-----------------------------------------------------------------
27+
1628
## [v0.20.0 (2023-09-12)](https://github.com/ArtARTs36/php-merge-request-linter/compare/0.19.1..0.20.0)
1729

1830
### Added
@@ -23,6 +35,8 @@ This file contains changelogs.
2335

2436
[💾 Assets](https://github.com/ArtARTs36/php-merge-request-linter/releases/tag/0.20.0)
2537

38+
-----------------------------------------------------------------
39+
2640
## [v0.19.1 (2023-09-12)](https://github.com/ArtARTs36/php-merge-request-linter/compare/0.19.0..0.19.1)
2741

2842
### Added

Makefile

+6-2
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,7 @@ check: deps-check
110110
composer stat-analyse
111111
composer deptrac
112112
composer test
113+
make security-check
113114

114115
info: ## Run "mr-linter info" on docker
115116
docker run \
@@ -134,7 +135,7 @@ lint-docker: docker-build
134135
--entrypoint "composer" \
135136
artarts36/merge-request-linter:testing "lint"
136137

137-
lint-fix-docker: docker-build
138+
lint-fix: docker-build
138139
docker run \
139140
--rm \
140141
--volume ./:/app/ \
@@ -171,9 +172,12 @@ deptrac: docker-build ## Run deptrac
171172
--entrypoint "composer" \
172173
artarts36/merge-request-linter:testing "deptrac"
173174

174-
check-docker: lint-docker stat-analyse test deptrac
175+
check-docker: lint-docker stat-analyse test deptrac security-check
175176

176177
build-phar: ## Build PHAR
177178
composer install --no-interaction --no-dev --prefer-dist --optimize-autoloader
178179
cd dev/build/ && composer install
179180
./dev/build/vendor/bin/box compile
181+
182+
security-check: ## Check security
183+
docker run --rm -it -w /app -v ./:/app pplotka/local-php-security-checker-github-actions --format=ansi

composer.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
],
1111
"require": {
1212
"php": "^8.2",
13-
"artarts36/str": "^2.6.0",
13+
"artarts36/str": "^2.7.0",
1414
"symfony/console": "^4.0 | ^5.0 | ^6.0",
1515
"guzzlehttp/psr7": "^2",
1616
"artarts36/local-file-system": "^0.1.2",

composer.lock

+7-7
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

docs/Builder/ConfigJsonSchema/OperatorSchemaArrayGenerator.php

+6
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,9 @@
3030
use ArtARTs36\MergeRequestLinter\Application\Condition\Evaluators\Strings\Cases\IsUpperCaseEvaluator;
3131
use ArtARTs36\MergeRequestLinter\Application\Condition\Evaluators\Strings\ContainsEvaluator;
3232
use ArtARTs36\MergeRequestLinter\Application\Condition\Evaluators\Strings\ContainsLineEvaluator;
33+
use ArtARTs36\MergeRequestLinter\Application\Condition\Evaluators\Strings\ContainsNumberEvaluator;
3334
use ArtARTs36\MergeRequestLinter\Application\Condition\Evaluators\Strings\EndsEvaluator;
35+
use ArtARTs36\MergeRequestLinter\Application\Condition\Evaluators\Strings\IsNumberEvaluator;
3436
use ArtARTs36\MergeRequestLinter\Application\Condition\Evaluators\Strings\LengthMaxEvaluator;
3537
use ArtARTs36\MergeRequestLinter\Application\Condition\Evaluators\Strings\LengthMinOperator;
3638
use ArtARTs36\MergeRequestLinter\Application\Condition\Evaluators\Strings\Markdown\ContainsHeadingEvaluator;
@@ -74,6 +76,8 @@ class OperatorSchemaArrayGenerator
7476
LinesMaxEvaluator::class,
7577
ContainsLineEvaluator::class,
7678
NotStartsAnyEvaluator::class,
79+
IsNumberEvaluator::class,
80+
ContainsNumberEvaluator::class,
7781
],
7882
Str::class => [
7983
EqualsEvaluator::class,
@@ -97,6 +101,8 @@ class OperatorSchemaArrayGenerator
97101
LinesMaxEvaluator::class,
98102
ContainsLineEvaluator::class,
99103
NotStartsAnyEvaluator::class,
104+
IsNumberEvaluator::class,
105+
ContainsNumberEvaluator::class,
100106
],
101107
Set::class => [
102108
CountMinEvaluator::class,

docs/conditions.md

+2
Original file line numberDiff line numberDiff line change
@@ -49,4 +49,6 @@ Currently is available that operators:
4949
| containsHeading5 | Check if a markdown-string contains a heading. | string |
5050
| containsHeading6 | Check if a markdown-string contains a heading. | string |
5151
| notIntersect | Check that the array does not intersect with the user array. | array of strings |
52+
| isNumber | Check if value are number. | boolean |
53+
| containsNumber | Check if value are contains number. | boolean |
5254

0 commit comments

Comments
 (0)