-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add more SDR record types #18
Conversation
0e2f407
to
17966b8
Compare
@richardstephens I've done some extra tinkering on this, and landed on just putting the relevant things in traits instead. I think it's good enough (for now, at least), since most of the relevant stuff is captured. I also added more sophisticated parser errors for all items, which should bubble up correctly now :) If you have time, I would appreciate it if you could put some eyes on it :) |
866618f
to
0dfaec6
Compare
Looks good. Our fork has fallen a bit behind, so I'll make time over the next couple of days to bring it up to latest main + these changes and check that everything still works. Thanks for making progress on my changes 😄 |
7ff7907
to
6db631a
Compare
This change also renames some structs to better reflect what they actually represent
Copy of #10.
Closes #10