Skip to content

DOC: Fixed typo in subsection heading#113

Merged
choldgraf merged 1 commit intoexecutablebooks:masterfrom
rossbar:doc
Mar 31, 2020
Merged

DOC: Fixed typo in subsection heading#113
choldgraf merged 1 commit intoexecutablebooks:masterfrom
rossbar:doc

Conversation

@rossbar
Copy link
Copy Markdown
Contributor

@rossbar rossbar commented Mar 31, 2020

Just a minor typo (I think) that I stumbled across while working my way through these docs. MyST-NB looks like it has some really nice features!

@chrisjsewell
Copy link
Copy Markdown
Member

Thanks @rossbar, I would merge yet though, since this is actually relevant to a wider conversation in #110, about what the name should be. You may wish to give your user perspective there 😄

@rossbar
Copy link
Copy Markdown
Contributor Author

rossbar commented Mar 31, 2020

I was just working my way through the issues now to try to get a handle on some of the main ideas/features (the one I'm most interested in is captions/labels for outputs, discussed in #72) and noticed that there had been discussion referencing glu: without the e - seems like there's already been some back-and-forth on the naming :)

No worries if the name isn't settled, it just stood out to me!

@choldgraf choldgraf merged commit 9e31271 into executablebooks:master Mar 31, 2020
@choldgraf
Copy link
Copy Markdown
Member

I think for now we should def merge in the name of "does this PR make the codebase better than before" :-) we can keep the discussion going a bit around the verbage but it may take some time to hash out the right word, and it'd be better if our docs were correct in the meantime, thanks @rossbar !

choldgraf pushed a commit that referenced this pull request Mar 31, 2020
choldgraf pushed a commit that referenced this pull request Mar 31, 2020
phaustin pushed a commit to phaustin/MyST-NB that referenced this pull request Apr 1, 2020
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.

3 participants