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

Issue 175: Updates to presentation of Reserved_Keys chapter -rebased on master #406

Merged
merged 3 commits into from May 26, 2022

Conversation

dsolt
Copy link
Contributor

@dsolt dsolt commented May 24, 2022

Rebased version of pr#392. This is the one we actually want to merge, but we need to make sure it is applying the same changes as 392.

@jjhursey
Copy link
Member

This PR is #392 but updated to match the current state of master.

Dave and I decided to create a new PR instead of rebasing the old PR because of the size of the changes. This will allow us to more easily view the changes side-by-side to make sure nothing is lost in the rebase.

@jjhursey jjhursey added Accepted as Stable ASC second vote passed. Accepted as Stable! Editorial labels May 25, 2022
@jjhursey jjhursey self-requested a review May 25, 2022 21:26
@jjhursey jjhursey added this to the PMIx v5 Standard milestone May 25, 2022
jjhursey
jjhursey previously approved these changes May 25, 2022
dsolt added 3 commits May 25, 2022 19:54
Clearer presentation of realms and partitioning keys into realms.
Also changed some labels to be more consistent.

Signed-off-by: dsolt@us.ibm.com
Signed-off-by: dsolt@us.ibm.com
- got rid of changes to PMIX_LOCAL_PROCS because the entire description
  was found to be inaccurate and will be addressed in a separate PR.
- remove <cr> in middle of a list
- retrieved retrieved -> retrieved
- a processes own -> a process' own
- fix refapi to PMIx_Tool_init -> PMIx_tool_init

Signed-off-by: dsolt@us.ibm.com
@jjhursey
Copy link
Member

I rebased this on to the current master after merging PR #405

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted as Stable ASC second vote passed. Accepted as Stable! Editorial
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants