-
Notifications
You must be signed in to change notification settings - Fork 2
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 metadata output #105
Add metadata output #105
Conversation
This should help us save useful metadata into the metadata file. The current code saved nothing.
Thank you for your contribution @natemcintosh 🚀! Your pkgdown-site is ready for download 👉 here 👈! |
Codecov ReportAll modified and coverable lines are covered by tests ✅ Additional details and impacted files📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I haven't run this code but it looks like other have -- thanks for the close reviews @amondal2 and @zsusswein. If you can get the tests to pass I think this looks good!
It was originally put there by copilot, and somehow I missed that it was there the whole time. This should have happened in the Config class PR. Not sure how it got through there.
This should help us save useful metadata into the metadata file. The current code saved nothing.