Skip to content

feat(rfc_tools): add RFC frontmatter and file markdown models - #8

Open
jtmcdole wants to merge 3 commits into
pr-2-taxonomy-clientfrom
pr-3-frontmatter-models
Open

feat(rfc_tools): add RFC frontmatter and file markdown models#8
jtmcdole wants to merge 3 commits into
pr-2-taxonomy-clientfrom
pr-3-frontmatter-models

Conversation

@jtmcdole

@jtmcdole jtmcdole commented Sep 4, 2026

Copy link
Copy Markdown
Member

PR 3 of 7 for testing and automation

RfcFile and RfcFrontmatter are the core models of parsing and validating.

  • rfc_file: extracting AAA.NNNN from files, markdown level 1 heading parser, and useful extensions to int.
  • rfc_frontmatter: yaml schema validation and error feedback

Stack created with GitHub Stacks CLIGive Feedback 💬

@jtmcdole
jtmcdole force-pushed the pr-3-frontmatter-models branch 2 times, most recently from 9781ddb to c5ac319 Compare September 4, 2026 17:13
@jtmcdole
jtmcdole marked this pull request as ready for review September 4, 2026 17:16
@jtmcdole
jtmcdole requested review from Piinks and zanderso September 4, 2026 17:26
@jtmcdole
jtmcdole force-pushed the pr-3-frontmatter-models branch from c5ac319 to f84b206 Compare September 4, 2026 19:33

@zanderso zanderso left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure since the frontmatter isn't too long, but would it be helpful to include file line numbers in the error messages?

Comment thread lib/src/models/rfc_file.dart Outdated
Comment thread lib/src/models/rfc_file.dart Outdated
Comment thread lib/src/models/rfc_file.dart Outdated
Comment thread lib/src/models/rfc_file.dart Outdated
Comment thread lib/src/models/rfc_frontmatter.dart Outdated
@jtmcdole
jtmcdole force-pushed the pr-3-frontmatter-models branch from f84b206 to 1e8f7aa Compare September 4, 2026 22:35
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