Skip to content

Commit 398d3ff

Browse files
sstoksstok
committed
Workflow improvements
- Use ^ instead of >= for Php version constraint - Split workflow files and use Rollerscapes Inspector for CS - Update issue templates - Don't cache vcs in workflow - Allow Symfony 8
1 parent 91d9b6c commit 398d3ff

13 files changed

Lines changed: 280 additions & 173 deletions

‎.github/ISSUE_TEMPLATE.md‎

Lines changed: 0 additions & 12 deletions
This file was deleted.

‎.github/ISSUE_TEMPLATE/1_Bug_report.md‎

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
---
2+
name: "\U0001F41B Bug Report"
3+
about: Report errors and problems
4+
title: "[bug] "
5+
labels: Potential Bug
6+
assignees: ''
7+
8+
---
9+
10+
**Description**
11+
<!-- A clear and concise description of the problem. -->
12+
13+
**How to reproduce**
14+
<!-- Code and/or config needed to reproduce the problem. -->
15+
16+
**Possible Solution**
17+
<!--- Optional: only if you have suggestions on a fix/reason for the bug -->
18+
19+
**Additional context**
20+
<!-- Optional: any other context about the problem: log messages, screenshots, etc. -->

‎.github/ISSUE_TEMPLATE/2_Feature_request.md‎

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
---
2+
name: "\U0001F680 Feature Request"
3+
about: "I have a suggestion (and may want to implement it \U0001F642)!"
4+
title: "[Feature] "
5+
labels: Feature
6+
assignees: ''
7+
8+
---
9+
10+
**Description**
11+
<!-- A clear and concise description of the new feature. -->
12+
13+
**Example**
14+
<!-- A simple example of the new feature in action (include PHP code, YAML config, etc.)
15+
If the new feature changes an existing feature, include a simple before/after comparison. -->

‎.github/ISSUE_TEMPLATE/3_Support_question.md‎

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
---
2+
name: 👩‍🏫 Support Question
3+
about: Questions about using this library
4+
labels: Question / Support
5+
6+
---
7+
8+
**Description**
9+
<!-- A clear and concise description of the problem. Also include what you tried, and any code
10+
snippets if you have them.
11+
12+
Please be patient, it may take some to get a response. Thanks.
13+
-->

‎.github/ISSUE_TEMPLATE/4_Security_issue.md‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
---
2+
name: ⛔ Security Issue
3+
about: Report security issues and problems (PLEASE DON'T DISCLOSE SECURITY-RELATED ISSUES PUBLICLY)
4+
5+
---
6+
7+
⚠ PLEASE DON'T DISCLOSE SECURITY-RELATED ISSUES PUBLICLY, SEE BELOW.
8+
9+
If you have found a security issue in this project, please send the details to
10+
security [at] rollerscapes.net and don't disclose it publicly until we can provide a
11+
fix for it.
12+
13+
**Note:** Please don't blindly send reports about automated tools, make sure the
14+
reported issue is in fact exploitable. Thanks.

‎.github/PULL_REQUEST_TEMPLATE.md‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,11 @@
55
| BC breaks? | yes/no
66
| Deprecations? | yes/no
77
| Fixed tickets | #... <!-- #-prefixed issue number(s), if any -->
8-
| License | MIT
8+
| License | MPL-2.0
99

1010
<!--
1111
- Please fill in this template according to the PR you're about to submit.
1212
Provide additional information in your description, not the questioner table.
1313
- Replace this comment by a description of what your PR is solving.
14+
- Do not replace the License, we only accept MPL-2.0 for this library.
1415
-->

‎.github/composer-config.json‎

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
{
2+
"config": {
3+
"cache-vcs-dir": "/dev/null",
4+
"platform-check": false,
5+
"preferred-install": {
6+
"*": "dist"
7+
},
8+
"allow-plugins": {
9+
"ergebnis/composer-normalize": true,
10+
"symfony/flex": true
11+
}
12+
}
13+
}

‎.github/workflows/ci.yaml‎

Lines changed: 0 additions & 158 deletions
This file was deleted.

‎.github/workflows/composer-validate.yaml‎

Lines changed: 51 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,51 @@
1+
name: Composer Validate
2+
3+
on:
4+
push:
5+
paths:
6+
- 'composer.json'
7+
pull_request:
8+
paths:
9+
- 'composer.json'
10+
11+
concurrency:
12+
group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
13+
cancel-in-progress: true
14+
15+
permissions:
16+
contents: read
17+
18+
jobs:
19+
composer-sync:
20+
name: 'Composer validation'
21+
runs-on: ubuntu-24.04
22+
env:
23+
php-version: '8.4'
24+
25+
steps:
26+
-
27+
name: Setup PHP
28+
uses: shivammathur/setup-php@v2
29+
with:
30+
php-version: ${{ env.php-version }}
31+
ini-values: "memory_limit=-1"
32+
coverage: none
33+
34+
-
35+
name: Checkout target branch
36+
uses: actions/checkout@v6
37+
38+
-
39+
name: 'Install dependencies'
40+
run: |
41+
COMPOSER_HOME="$(composer config home)"
42+
([ -d "$COMPOSER_HOME" ] || mkdir "$COMPOSER_HOME") && cp .github/composer-config.json "$COMPOSER_HOME/config.json"
43+
composer global require -q "ergebnis/composer-normalize"
44+
composer install --no-progress
45+
46+
-
47+
name: 'Normalized composer.json'
48+
run: |
49+
echo "composer.json"
50+
composer validate
51+
composer normalize

‎.github/workflows/inspector-bot.yaml‎

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
name: CS
2+
3+
on:
4+
pull_request:
5+
6+
permissions:
7+
contents: read
8+
9+
jobs:
10+
call-inspector-bot:
11+
name: InspectorBot
12+
uses: rollerscapes/inspector-bot/.github/workflows/inspector-bot.yml@main
13+
with:
14+
package: RollerworksSplitToken
15+
check_license: false # We use MPL-2.0 and this not supported by the bot yet
16+

0 commit comments

Comments
 (0)