-
Notifications
You must be signed in to change notification settings - Fork 532
API reports and review
DRAFT: in progress
Some Fluid packages are in various states of readiness and supportability. As we formally lock down on APIs, we need a way to trigger review of any changes to the public API surface. That's where API reports come in.
API reports are Markdown files comprised mostly of a large block of pseudocode that concisely summarize the API signatures for a given package. These reports are committed to Git in the api-report
folder at the root of the repo.
During the local build process, any packages with API reporting enabled will check if there are API changes, then automatically make those changes locally. You can then include that change in your PR so that the API change can be reviewed.
If an API change is made without the corresponding report change, then the CI build will fail.
- Change the package's
api-extractor.json
to extend@fluidframework/build-common/api-extractor-common-report.json
. - Run a local build and commit the files generated in the
api-report
folder. - Open a PR with the changes.
- Once merged, you're done!
This wiki is focused on contributing to the Fluid Framework codebase.
For information on using Fluid Framework or building applications on it, please refer to fluidframework.com.
- Submitting Bugs and Feature Requests
-
Contributing to the Repo
- Repo Basics
- Common Workflows and Patterns
- Managing dependencies
- Client Code
- Server Code
- PR Guidelines
- CI Pipelines
- Breaking vs Non-Breaking Changes
- Branches, Versions, and Releases
- Compatibility & Versioning
- Testing
- Debugging
- npm package scopes
- Maintaining API support levels
- Developer Tooling Maintenance
- API Deprecation
- Working with the Website (fluidframework.com)
- Coding Guidelines
- Documentation Guidelines
- CLA