Skip to content

Conversation

@mkannwischer
Copy link
Contributor

Resolves #105

Signed-off-by: Matthias J. Kannwischer <matthias@kannwischer.eu>
@mkannwischer mkannwischer requested a review from a team as a code owner November 7, 2024 13:44
@clowarden
Copy link

clowarden bot commented Nov 7, 2024

Validation succeeded

✅ The proposed configuration changes are valid!

Configuration changes

Directory

  • team pqcp-c-aarch64-contributors has been removed
  • team pqcp-c-aarch64-admin has been removed
  • team pqcp-c-aarch64-maintainers has been removed
  • team pqcp-native-contributors has been added
    • Maintainers
      • hanno-becker
      • mkannwischer
    • Members
      • cothan
      • potsrevennil
      • rod-chapman
  • team pqcp-native-maintainers has been added
    • Maintainers
      • mkannwischer
    • Members
      • hanno-becker
  • team pqcp-native-admin has been added
    • Maintainers
      • mkannwischer
      • planetf1

Github

  • repository mlkem-native has been added (visibility: public)
    • Teams
      • bots: admin
      • pqcp-native-admin: admin
      • pqcp-native-contributors: write
      • pqcp-native-maintainers: maintain
      • pqcp-tsc: read

🔸 Please review the changes detected as they will be applied immediately once this PR is merged 🔸

Copy link
Contributor

@planetf1 planetf1 left a comment

Choose a reason for hiding this comment

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

LGTM

There are also the old mlkem-c-generic group definitions. Can either add here, or I'm happy to do another PR after you've done this one.

Are you going to do the actual github repo rename @mkannwischer ?

@ryjones
Copy link
Contributor

ryjones commented Nov 7, 2024

merge and then sit tight

@ryjones ryjones merged commit cd3ea11 into pq-code-package:main Nov 7, 2024
3 checks passed
@clowarden
Copy link

clowarden bot commented Nov 7, 2024

Reconciliation completed

✅ The reconciliation completed successfully and all changes have been applied across the services!

Changes applied

Github

  • team pqcp-c-aarch64-maintainers has been removed
  • team pqcp-c-aarch64-admin has been removed
  • team pqcp-c-aarch64-contributors has been removed
  • team pqcp-native-contributors has been added
    • Maintainers
      • hanno-becker
      • mkannwischer
    • Members
      • cothan
      • potsrevennil
      • rod-chapman
  • team pqcp-native-maintainers has been added
    • Maintainers
      • mkannwischer
    • Members
      • hanno-becker
  • team pqcp-native-admin has been added
    • Maintainers
      • mkannwischer
      • planetf1
  • team pqcp-native-maintainers has been added to repository mlkem-native (role: maintain)
  • team pqcp-native-admin has been added to repository mlkem-native (role: admin)
  • team pqcp-native-contributors has been added to repository mlkem-native (role: write)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Rename mlkem-c-aarch64 to mlkem-native

4 participants