Add coding guideline for optional fields.

[mackjmr]

Description

Specify to use pointers to represent optional fields. This is a follow up to: #9478.

Link to tracking issue

Fixes #9478

Testing

Documentation

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.54%. Comparing base (2d25853) to head (63028be).
Report is 181 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #11342   +/-   ##
=======================================
  Coverage   91.54%   91.54%           
=======================================
  Files         428      428           
  Lines       20230    20230           
=======================================
  Hits        18519    18519           
  Misses       1337     1337           
  Partials      374      374           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[songy23]

LGTM

[mx-psi]

After discussing with Evan, we may want to do a PoC of the hook that would be needed to support Optional to have a more concrete idea of the complexity we would be adding with this alternative.

[mackjmr]

@evan-bradley @yurishkuro @mx-psi I've taken a stab at this in #11409, can you please take a look ? I may be missing some edge cases.

[github-actions]

This PR was marked stale due to lack of activity. It will be closed in 14 days.

[github-actions]

Closed as inactive. Feel free to reopen if this PR is still being worked on.