CycloneDX / CycloneDX/cyclonedx-php-library

[PHP8.1] use `readonly` properties

Open
#141 1 comment 0 reactions 1 assignee Claimed by @jkowalleck View on GitHub
breaking change
Dominant language
PHP
Stars
13
Forks
0
PR merge metrics
No merged PRs in 30d

Description

currently readonly properties are private, annotated as `@readonly` and available via a getter.

since php8.1 native support for readonly properties exists.

# proposal
- [ ] remove the getter
- [ ] make the property public
- [x] make the property readonly - via #208

## attention

- there are issues when it comes to modifications in `__clone()` - -therefore, not fully rolled out
- there are extended works needed to make the test cases aware ...
- failed try: #207

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.