Skip to content
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

[WIP]: Updating MPI Tutorials #17

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[WIP]: Updating MPI Tutorials #17

wants to merge 1 commit into from

Conversation

nhanford
Copy link

@nhanford nhanford commented Dec 21, 2023

I am going through and updating our MPI tutorials, first with proofreading and correctness in mind, and then checking all the examples against those in the MPI Standard.

This is still work in progress and is not yet ready for merging.

Copy link
Member

@IanLee1521 IanLee1521 left a comment

Choose a reason for hiding this comment

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

Looks good to me so far!

@IanLee1521 IanLee1521 requested a review from xorJane January 11, 2024 04:53
@nhanford
Copy link
Author

In particular, I was wondering if the fortran syntax highlighting is rendering correctly, but I'm not sure how to get CI working to test this...

@xorJane xorJane marked this pull request as ready for review February 16, 2024 21:23
@xorJane
Copy link
Member

xorJane commented Feb 16, 2024

Hey @nhanford is this one still a WIP? If you're just unsure about whether the fortran syntax highlighting will work/how to test in CI, I think it's okay to test via merging. I doubt that will be a breaking change and we could always revert.

@lostmyhat lostmyhat self-requested a review March 25, 2024 00:03
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