Skip to content

Update metadata.authors & USAGE #289

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

Merged
merged 2 commits into from
Dec 18, 2019
Merged

Conversation

dhimmel
Copy link
Member

@dhimmel dhimmel commented Dec 18, 2019

metadata.yaml should now use authors instead of author_info.

Update USAGE to mention delete-me.

@@ -179,7 +182,7 @@ The following YAML shows the supported key–value pairs for an author:
```yaml
github: dhimmel # strongly suggested
name: Daniel S. Himmelstein # mandatory
initials: DSH # strongly suggested
initials: DSH # optional
Copy link
Member Author

Choose a reason for hiding this comment

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

initials are not used anywhere by default, so seems excessive to call them "strongly suggested"

Copy link
Member

Choose a reason for hiding this comment

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

Agreed. I think that was a remnant from deep review where we were trying to automate things like funding statements. We haven't needed them in more modern forms of Manubot.

@@ -12,6 +12,9 @@ Markdown files are identified by their `.md` extension and ordered according to
For basic formatting, check out the [CommonMark Help](https://commonmark.org/help/) page for an introduction to the formatting options provided by standard markdown.
In addition, Manubot supports an extended version of markdown, tailored for scholarly writing, which includes [Pandoc's Markdown](https://pandoc.org/MANUAL.html#pandocs-markdown) and the extensions discussed below.

The `content/02.delete-me.md` file in the Rootstock repository shows many of the elements and formatting options supported by Manubot.
See the [raw markdown](https://gitlab.com/manubot/rootstock/blob/master/content/02.delete-me.md#L) in this file and compare it to the [rendered manuscript](https://manubot.github.io/rootstock/).
Copy link
Member Author

Choose a reason for hiding this comment

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

Meant to help address #288 (comment).

CC @Nebucatnetzer and @vincerubinetti

Copy link
Member Author

Choose a reason for hiding this comment

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

Using GitLab link to raw markdown as a workaround for isaacs/github#297. Will be slightly confusing, but not sure what is best.

@dhimmel dhimmel merged commit 37b3378 into manubot:master Dec 18, 2019
dhimmel added a commit that referenced this pull request Dec 18, 2019
This build is based on
37b3378.

This commit was created by the following Travis CI build and job:
https://travis-ci.com/manubot/rootstock/builds/141642153
https://travis-ci.com/manubot/rootstock/jobs/268654692

The full commit message that triggered this build is copied below:

Update metadata.authors & USAGE

Merges #289

* Update metadata.yaml to use 'authors'
* Mention delete-me.md in USAGE
dhimmel added a commit that referenced this pull request Dec 18, 2019
This build is based on
37b3378.

This commit was created by the following Travis CI build and job:
https://travis-ci.com/manubot/rootstock/builds/141642153
https://travis-ci.com/manubot/rootstock/jobs/268654692

The full commit message that triggered this build is copied below:

Update metadata.authors & USAGE

Merges #289

* Update metadata.yaml to use 'authors'
* Mention delete-me.md in USAGE
@dhimmel dhimmel deleted the author_info_dep branch December 19, 2019 15:48
ploegieku added a commit to ploegieku/2023-functional-homology-paper that referenced this pull request Aug 6, 2024
Merges manubot/rootstock#289

* Update metadata.yaml to use 'authors'
* Mention delete-me.md in USAGE
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