Skip to content

Allow ignoring unknown metadata#360

Merged
TethysSvensson merged 2 commits into
planus-org:mainfrom
parched:unknown-metadata
Jul 11, 2026
Merged

Allow ignoring unknown metadata#360
TethysSvensson merged 2 commits into
planus-org:mainfrom
parched:unknown-metadata

Conversation

@parched

@parched parched commented May 6, 2026

Copy link
Copy Markdown
Contributor

Builds on #312

When using flatbuffers to share data between different languages
that have different generators there can be attributes that are specific
to one particular generator.
E.g., https://github.com/jamescourtney/FlatSharp/wiki/FBS-Annotations#flatsharp-fbs-extensions
In this case, the other generators should just ignore these attributes.

Note: implemented using AI

Checklist

  • Updated CHANGELOG.md with relevant changes
  • Added tests for any new/fixed functionality
  • Added/updated documentation for new/changed code
  • Checked that README.md still makes sense (and updated it if necessary)

@TethysSvensson

Copy link
Copy Markdown
Collaborator

This looks good to me. Would you mind rebasing it on top of main?

parched added 2 commits July 11, 2026 21:12
When using flatbuffers to share data between different languages
that have different generators there can be attributes that are specific
to one particular generator.
E.g., https://github.com/jamescourtney/FlatSharp/wiki/FBS-Annotations#flatsharp-fbs-extensions
In this case, the other generators should just ignore these attributes.

Note: implemented using AI
@parched parched force-pushed the unknown-metadata branch from e68ea2e to 96838e1 Compare July 11, 2026 09:16
@parched

parched commented Jul 11, 2026

Copy link
Copy Markdown
Contributor Author

@TethysSvensson Sure thing, done.

@TethysSvensson TethysSvensson merged commit f8d5b99 into planus-org:main Jul 11, 2026
3 checks passed
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