Allow to create contract-based specifications with some local overrides
WHY are these changes introduced?
Refactors the createContractBasedModuleSpecification function to accept a single object parameter instead of multiple arguments, making the API more flexible and consistent.
WHAT is this pull request doing?
- Modifies
createContractBasedModuleSpecificationto accept a configuration object withidentifier,appModuleFeatures, andbuildConfigproperties - Updates all calls to this function throughout the codebase
- Simplifies the channel specification implementation by leveraging the contract-based module specification
How to test your changes?
- Run the extension specification tests to ensure they pass
- Create a new channel extension to verify the specification works correctly
- Verify that existing extensions using contract-based module specifications continue to function properly
Measuring impact
How do we know this change was effective? Please choose one:
- [x] n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
- [ ] Existing analytics will cater for this addition
- [ ] PR includes analytics changes to measure impact
Checklist
- [x] I've considered possible cross-platform impacts (Mac, Linux, Windows)
- [x] I've considered possible documentation changes
- #6628
π (View in Graphite) main
This stack of pull requests is managed by Graphite. Learn more about stacking.
We detected some changes at packages/*/src and there are no updates in the .changeset.
If the changes are user-facing, run pnpm changeset add to track your changes and include them in the next release CHANGELOG.
[!CAUTION] DO NOT create changesets for features which you do not wish to be included in the public changelog of the next CLI release.
Coverage report
St.:grey_question: |
Category | Percentage | Covered / Total |
|---|---|---|---|
| π‘ | Statements | 79.25% (+0.02% πΌ) |
13582/17139 |
| π‘ | Branches | 73.14% (+0.03% πΌ) |
6632/9068 |
| π‘ | Functions | 79.41% (+0.04% πΌ) |
3505/4414 |
| π‘ | Lines | 79.6% (+0.02% πΌ) |
12828/16116 |
Show files with reduced coverage π»
St.:grey_question: |
File | Statements | Branches | Functions | Lines |
|---|---|---|---|---|---|
| π‘ | ... / specification.ts |
68.52% (-0.57% π») |
74.36% (+1.19% πΌ) |
76.47% (-1.31% π») |
68.09% (-0.66% π») |
Test suite run success
3356 tests passing in 1372 suites.
Report generated by π§ͺjest coverage report action from 9ae216cec1ebf904db2266f3a0a0fb41fdb57f6a