ix
ix copied to clipboard
refactor(core): replace internal comments with annotations
๐ก What is the current behavior?
GitHub Issue Number: #<ISSUE NUMBER>
๐ What is the new behavior?
๐ Checklist
A pull request can only be merged if all of these conditions are met (where applicable):
- [ ] ๐ฆฎ Accessibility (a11y) features were implemented
- [ ] ๐บ๏ธ Internationalization (i18n) - no hard coded strings
- [ ] ๐ฒ Responsiveness - components handle viewport changes and content overflow gracefully
- [x] ๐ Documentation was reviewed/updated (
pnpm run docs) - [ ] ๐งช Unit tests were added/updated and pass (
pnpm test) - [ ] ๐ธ Visual regression tests were added/updated and pass (Guide)
- [ ] ๐ง Static code analysis passes (
pnpm lint) - [x] ๐๏ธ Successful compilation (
pnpm build, changes pushed)
๐จโ๐ป Help & support
๐ฆ Changeset detected
Latest commit: 078d7c85f6e61fa08bbfd05890b98e0c0974bc54
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 5 packages
| Name | Type |
|---|---|
| @siemens/ix-angular | Major |
| @siemens/ix-aggrid | Major |
| @siemens/ix-react | Major |
| @siemens/ix | Major |
| @siemens/ix-vue | Major |
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
Report of strictNullChecks (strictPropertyInitialization) check
Typescript check
You decreased the total number of errors in PR branch by 3 ๐
ESLint check
Total ESLint warnings are the same in both branches