-
Notifications
You must be signed in to change notification settings - Fork 1
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
Pull in changes from 'upstream' #39
base: main
Are you sure you want to change the base?
Conversation
Oops no payload
Delete TODO.md
…to upstream-template
@CallumWalley 90% looks good however it looks to be broken on the code blocks as can be seen here |
Basic checks looks good to me |
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.
LGTM
Sorry, meant to do this days ago, got sidetracked.
Let me know if you have any questions, changes, etc.
Changes:
.pages.yml
files now do the same function assummary.md
, more details. I left most of the explicitly set names in place, but it's best to use the filename when possible.assets/stylesheets
(seems slightly more common and is what is used on other sites).requirements.in
for use withpip compile
.