Skip to content
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 metadata from redata to readme #277

Merged
merged 13 commits into from
Feb 5, 2025

Conversation

HafeezOJ
Copy link
Contributor

Description

See #

Documentation Update

  • I have updated README.md and other relevant documentation
  • No documentation update is needed

Implementation Notes

This PR adds more metadata to the README template in LDCOOLP. The metadata added by this PR includes funding information and related materials, they will be included in README.txt generated by LDCOOLP if the metadata are available for an item in Figshare.

A jinja template macro is also included to provide the general name for a funder when applicable. The macro setting/template is included in the configuration file; placeholder holders have been included in the default configuration template.

@HafeezOJ HafeezOJ linked an issue Jan 16, 2025 that may be closed by this pull request
1 task
@HafeezOJ HafeezOJ assigned HafeezOJ and unassigned HafeezOJ Jan 16, 2025
@HafeezOJ HafeezOJ requested a review from zoidy January 16, 2025 22:50
Copy link
Collaborator

@zoidy zoidy left a comment

Choose a reason for hiding this comment

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

Readme changes look good. Please revert changes related to the linter failure and create a separate issue to handle dependency updates (there are a couple of other PRs from the dependency bot that could be included)

@HafeezOJ
Copy link
Contributor Author

HafeezOJ commented Feb 4, 2025

The changes related to the linter failure have been reverted.

@zoidy
Copy link
Collaborator

zoidy commented Feb 5, 2025

Merging PR, linter failures will be addressed separately

@zoidy zoidy merged commit 83e5516 into master Feb 5, 2025
0 of 6 checks passed
@zoidy zoidy deleted the 276-add-more-metadata-from-redata-to-readme branch February 5, 2025 13:17
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.

Add more metadata from ReDATA to README
2 participants