Skip to content

Commit 8363dc6

Browse files
authored
Exclude generated Proto code from auto formatting (nuwave#2672)
1 parent d9fb0ee commit 8363dc6

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

48 files changed

+1405
-1791
lines changed

.github/workflows/proto.yml

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,6 @@ jobs:
1515
with:
1616
ref: ${{ github.head_ref }}
1717

18-
- uses: shivammathur/setup-php@v2
19-
with:
20-
coverage: none
21-
extensions: mbstring
22-
php-version: 8.3
23-
24-
- run: composer install --no-interaction --no-progress --no-suggest
25-
2618
- uses: bufbuild/buf-setup-action@v1
2719

2820
- run: |
@@ -31,8 +23,6 @@ jobs:
3123
mv proto-tmp/Nuwave/Lighthouse/Tracing/FederatedTracing/Proto src/Tracing/FederatedTracing/Proto
3224
rm -rf proto-tmp
3325
34-
- run: vendor/bin/php-cs-fixer fix src/Tracing/FederatedTracing/Proto
35-
3626
- run: git pull
3727

3828
- uses: stefanzweifel/git-auto-commit-action@v5

.php-cs-fixer.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
$finder = PhpCsFixer\Finder::create()
44
->in(__DIR__)
55
->name('*.php')
6+
->notPath('src/Tracing/FederatedTracing/Proto')
67
->notPath('vendor')
78
->notPath('phpstan-tmp-dir')
89
->ignoreDotFiles(false)

Makefile

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,4 +87,3 @@ proto:
8787
sudo chown --recursive "$(shell id -u):$(shell id -g)" proto-tmp
8888
mv proto-tmp/Nuwave/Lighthouse/Tracing/FederatedTracing/Proto src/Tracing/FederatedTracing/Proto
8989
rm -rf proto-tmp
90-
$(MAKE) php-cs-fixer

src/Schema/Directives/RelationDirectiveHelpers.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ protected function makeBuilderDecorator(mixed $root, array $args, GraphQLContext
4444
$this->scopes(),
4545
/**
4646
* Sometimes overridden to use a different model than the usual root.
47+
*
4748
* @see \Nuwave\Lighthouse\Execution\ModelsLoader\PaginatedModelsLoader::loadRelatedModels
4849
* @see \Tests\Integration\Schema\Directives\BuilderDirectiveTest::testCallsCustomBuilderMethodOnFieldWithSpecificModel
4950
*/

src/Tracing/FederatedTracing/Proto/ContextualizedQueryLatencyStats.php

Lines changed: 29 additions & 29 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)