-
Notifications
You must be signed in to change notification settings - Fork 0
Added AFS #31
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
base: main
Are you sure you want to change the base?
Added AFS #31
Conversation
| urls: | ||
| - https://afs.princeton.edu/undergraduate-opportunities/certificate-african-studies | ||
| description: The program offers classes in Africa’s political, economic, and social history; | ||
| built environments and urban geographies; ecology, genetic diversity, and epidemiological concerns. |
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.
Make sure to indent these lines
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.
These lines are part of the quote from the previous line, so they should be indented to show nesting.
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.
Even better, use one of YAML's block string formats
https://stackoverflow.com/questions/3790454/how-do-i-break-a-string-in-yaml-over-multiple-lines
bnehoran
left a comment
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.
Looks good! Just a few small things.
| code: AFS | ||
| degree: | ||
| urls: | ||
| - https://afs.princeton.edu/undergraduate-opportunities/certificate-african-studies |
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.
This line doesn't need to be indented. The dash that designates it as a list already counts as an indentation. Not sure what the official style guide says, but this keeps it consistent with the rest of the lists in this file.
| urls: | ||
| - https://afs.princeton.edu/undergraduate-opportunities/certificate-african-studies | ||
| description: The program offers classes in Africa’s political, economic, and social history; | ||
| built environments and urban geographies; ecology, genetic diversity, and epidemiological concerns. |
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.
These lines are part of the quote from the previous line, so they should be indented to show nesting.
| urls: | ||
| - https://afs.princeton.edu/undergraduate-opportunities/certificate-african-studies | ||
| description: The program offers classes in Africa’s political, economic, and social history; | ||
| built environments and urban geographies; ecology, genetic diversity, and epidemiological concerns. |
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.
Even better, use one of YAML's block string formats
https://stackoverflow.com/questions/3790454/how-do-i-break-a-string-in-yaml-over-multiple-lines
No description provided.