wol-soft / php-json-schema-model-generator

99%

master: 99%

LAST BUILD ON BRANCH claude/code-formatting-analysis-ipp4bf

branch: claude/code-formatting-analysis-ipp4bf

CHANGE BRANCH

x

Reset

• claude/code-formatting-analysis-ipp4bf
• 0.1.0
• 0.10.0
• 0.10.1
• 0.11.0
• 0.11.1
• 0.12.0
• 0.13.0
• 0.14.0
• 0.14.1
• 0.15.0
• 0.15.1
• 0.16.0
• 0.17.0
• 0.18.0
• 0.19.0
• 0.2.0
• 0.20.0
• 0.21.0
• 0.21.1
• 0.21.2
• 0.21.3
• 0.21.4
• 0.21.5
• 0.21.6
• 0.21.7
• 0.21.8
• 0.22.0
• 0.22.1
• 0.23.0
• 0.23.1
• 0.23.2
• 0.23.3
• 0.23.4
• 0.23.5
• 0.24.0
• 0.24.1
• 0.24.2
• 0.25.0
• 0.25.1
• 0.26.1
• 0.26.2
• 0.3.0
• 0.3.1
• 0.4.0
• 0.5.0
• 0.6.0
• 0.7.0
• 0.8.0
• 0.9.0
• 65-recursive-reference-resolving
• 70-filter-in-composition
• AddOptionToDefaultArraysToEmptyArray
• AdvancedFilter
• BaseProcessor
• BuilderClassPostProcessor
• CompositionValidationOnMutableObjects
• FormatValidation
• JsonSchemaDraftTestSuite
• MoveCI
• NestedObjectException
• OptimizeMergedProperties
• PatternProperties
• PerformanceImprovements
• PostProcessor
• PostProcessorHooks
• ReworkExceptions
• SchemaHooks
• SkipFilterAfterTransformingFilterIfTransformationFails
• ThrowExceptionFromNestedModel
• additionalProperties
• attributes
• bogus-loop-var
• booleanSchemas
• chore/phpunit-13-upgrade
• claude/comprehensiveformatting-arrays-ksd26j
• claude/exciting-cori-7aca14
• claude/issue-167-q6ebfv
• claude/issue-169-ti28cz
• claude/issue-171-b6066p
• claude/issue-172-yns270
• claude/library-output-generation-plan-fmo3bz
• claude/mr-74-review-adoption-3p8hdm
• claude/mr-93-typescript-postprocessor-8u1wwl
• claude/pr-151-review-oaznmm
• claude/pr-166-continuation-8dtt5e
• claude/pr-166-coverage-3hhtc2
• claude/ref-resolution-improvements-th04wg
• claude/remove-ext-json-requirement-6rewh4
• claude/schema-error-line-numbers-b76c62
• claude/type-guard-collision-eval-3c5l0n
• claude/validation-error-messages-c73hox
• debug
• drafts
• enumPostProcessor
• feat-required-and-optional-const-property
• feat/composition-required-promotion
• feature/class-name-generator-improvements
• feature/defs-keyword-support
• filter-composition-ordering
• fix-implicit-null-allowed
• fix-name-normalization
• fix/composed-schema-branch-defaults
• fix/enum-types-in-composition-branches
• fix/issue-101
• fix/issue-110
• fix/issue-129-enum-type-check-validator
• fix/issue-130-duplicate-ref-builder-methods
• fix/issue-98
• implicitNullableConfiguration
• improveRefResolving
• issue133
• issue72_nestedComposition
• issue77_dynamicProperty
• issue79_CombinedRefAndObjectDefinition
• jsonSchemaDraft7
• master
• modernizeCode
• optionalConst
• order-schema-files
• param
• patch-1
• patch-2
• php82tessting
• propertyinterface-bogus-signature
• ref-required-and-optional-property
• refImprovements
• renameRawModelDataInput
• supportNumericProperties
• typos
• unevaluated-properties
• useInNestedReferences

Build # 30326374406

Build Type

Pull #178

github

Committed by claude

Commit Message

Remove dead code and its explanatory comment from BuilderClass.phptpl

The @throws block for a builder setter's validators is unreachable code, not
just a documented limitation: BuilderClassPostProcessor unconditionally
strips every validator from builder properties via
filterValidators(fn() => false), so property.getValidators() can never be
non-empty in this template. Removed the block entirely instead of keeping a
dead branch alive "for whenever that changes" - if validators are ever
un-stripped, the @throws line can be reintroduced then, informed by whatever
that change actually needs.

Also drops the description-block comment explaining the standalone-if
choice - the pattern is now used pervasively across every class-emitting
template and no longer needs a per-callsite justification.

Pull Request Pull Request #178: Render generated code PSR-12 compatible

Coverage Stats

199 of 201 new or added lines in 23 files covered. (99.0%)

7176 of 7268 relevant lines covered (98.73%)

573.82 hits per line