FIDL Project Checklist

Changes to FIDL can often have a large number of downstream effects. This document provides checklists for items that should be considered for various types of FIDL related changes. Note that not all elements of the checklist will necessarily apply to every change, but they should at least be helpful for developers to double check that their changes are complete.

Change to the fidlc compiler

This checklist is for changes that affect FIDL itself, such as syntax or semantics changes.

  • Update documentation.
  • Update associated tools.
    • The FIDL formatter/linter and their associated TreeVisitors.
    • Editors and syntax highlighting.
  • Update FIDL files. For syntax changes, update existing FIDL source including:
    • Actual FIDL source files in tree (and out of tree, if required).
    • FIDL source specified as strings in test source code, such as the [compiler unit tests][compiler-test] and the fidlgen end-to-end tests.
    • FIDL snippets in documentation.
  • Ensure that interactions with attributes are handled.
    • For example, does the feature require a new placement?
  • Update integration tests
    • For larger language features, it may be worth adding test coverage in existing integration tests. For example, ensuring cross binding consistency through the compatibility tests, or transitionability through the source compatibility tests.
  • For parser changes specifically:
    • Update the span_tests to ensure that the parsed spans are correct, and not just that parsing succeeds.
  • For semantics changes:
    • Update the feature specific unittest, e.g. table_tests when making a modification to tables, or create a new test file.

Change to the JSON IR

Change to fidlgen_<lang>

  • Update the bindings docs if the generated code has changed
  • Update the bindings tutorials to reflect the current best practices and patterns.
  • Update any relevant [guides].
    • For example, when changing the memory allocation APIs in LLCPP, the LLCPP allocator guide should be updated.