Skip to content

#38 port cmake material#39

Open
martinjrobins wants to merge 4 commits into
mainfrom
i38-cmake
Open

#38 port cmake material#39
martinjrobins wants to merge 4 commits into
mainfrom
i38-cmake

Conversation

@martinjrobins

Copy link
Copy Markdown
Collaborator

Fixes #38

@fcooper8472

Copy link
Copy Markdown
Collaborator

@martinjrobins I've started making a few changes, e.g. splitting the expected output from the commands themselves.

I also have a preference for the codeblocks being directly copyable, so I've removed some of the dir$ decoration... but before I go through the rest, happy to hear your thoughts on those changes.

Re output of code blocks, I want to revisit this work:
OxfordRSE/gutenberg#40
which would allow some decoration including (e.g.) that the code represents expected output, rather than input.

@martinjrobins

Copy link
Copy Markdown
Collaborator Author

yup, agree that code blocks should be copyable, so happy with this

@alasdairwilson

Copy link
Copy Markdown
Collaborator

Is this PR still relevant?

@martinjrobins

martinjrobins commented Jun 4, 2025

Copy link
Copy Markdown
Collaborator Author

yes, been waiting for a review/changes by @fcooper8472....

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.

port cmake course

3 participants