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

Errata: Type of callback pmix_iof_cbfunc_t parameter payload should b… #486

Merged
merged 1 commit into from
Jan 25, 2024

Conversation

dsolt
Copy link
Contributor

@dsolt dsolt commented Jun 5, 2023

…e a pmix_byte_object_t *

Issue #421

@dsolt dsolt added the Errata label Jun 5, 2023
@dsolt dsolt added the RFC Request for Comment label Jun 5, 2023
@dsolt
Copy link
Contributor Author

dsolt commented Jun 5, 2023

Please use emoji reactions ON THIS COMMENT to indicate your position on this proposal.

  • You do not need to vote on every proposal
  • If you have no opinion, don't vote - that is also useful data
  • If you've already commented on this issue, please still vote so
    we know your current thoughts
  • Not all proposals solve exactly the same problem, so we may end
    up accepting proposals that appear to have some overlap
    This is not a binding majority-rule vote, but it will be a very
    significant input into the corresponding ASC decision.

Here are the meanings for the emojis:

  • Hooray or Rocket: I support this so strongly that I
    want to be an advocate for it
  • Heart: I think this is an ideal solution
  • Thumbs up: I'd be happy with this solution
  • Confused: I'd rather we not do this, but I can tolerate it
  • Thumbs down: I'd be actively unhappy, and may even consider
    other technologies instead
    If you want to explain in more detail, feel free to add another
    comment, but please also vote on this comment.

@jjhursey jjhursey added the Eligible Eligible for consideration by ASC label Jul 3, 2023
@jjhursey
Copy link
Member

PMIx ASC 3Q 2023 Voting results:

  • Yes: 6
  • No: 0
  • Abstain: 0
  • Result: Pass as Errata. Can be merged at the discretion of Co-Chairs/RMs.

@jjhursey jjhursey added the First Vote Passed ASC first vote passed label Jul 20, 2023
@abouteiller abouteiller changed the base branch from master to v4 January 11, 2024 10:46
@abouteiller abouteiller changed the base branch from v4 to master January 11, 2024 10:47
@abouteiller abouteiller self-requested a review January 11, 2024 10:49
@abouteiller abouteiller merged commit b5e6dc8 into pmix:master Jan 25, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Eligible Eligible for consideration by ASC Errata First Vote Passed ASC first vote passed RFC Request for Comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants