Skip to content

Refactor framework definitions#918

Merged
bocchino merged 4 commits intomainfrom
refactor-framework-defs
Feb 5, 2026
Merged

Refactor framework definitions#918
bocchino merged 4 commits intomainfrom
refactor-framework-defs

Conversation

@bocchino
Copy link
Collaborator

@bocchino bocchino commented Feb 5, 2026

  • Use maps instead of option variables. The resulting code is much shorter and less repetitive.
  • Move CheckFrameworkDefs up in CheckSemantics, so the results are available earlier.

@jwest115 Note the JSON analysis representation has changed.

Making a PR into #913 because #913 is backed up behind nasa/fprime#4673. Once the F Prime PR clears, we can merge #913 and then this PR. @LeStarch @thomas-bc FYI.

@bocchino bocchino requested a review from jwest115 February 5, 2026 05:10
@bocchino bocchino marked this pull request as ready for review February 5, 2026 05:10
Base automatically changed from issue-903-default-string-size to main February 5, 2026 22:29
@bocchino bocchino merged commit 8dd964c into main Feb 5, 2026
19 checks passed
@bocchino bocchino deleted the refactor-framework-defs branch February 5, 2026 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants