Skip to content

Conversation

@lukasjhan
Copy link
Member

Hi @ryjones

Requesting hdcx-js repo: openwallet-foundation/project-proposals#61

repo transferring will be sent to you soon

@clowarden
Copy link

clowarden bot commented Aug 12, 2025

Validation succeeded

✅ The proposed configuration changes are valid!

Configuration changes

Directory

  • team hdcx-js-maintainers has been added
    • Maintainers
      • lukasjhan
    • Members
      • aceshim
      • yunseorim1116

Github

  • repository hdcx-js has been added (visibility: public)
    • Teams
      • hdcx-js-maintainers: maintain

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

@clowarden
Copy link

clowarden bot commented Aug 12, 2025

Validation succeeded

✅ The proposed configuration changes are valid!

Configuration changes

Directory

  • team hdcx-js-maintainers has been added
    • Maintainers
      • lukasjhan
    • Members
      • aceshim
      • yunseorim1116

Github

  • repository hdcx-js has been added (visibility: public)
    • Teams
      • hdcx-js-maintainers: maintain

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

@lukasjhan
Copy link
Member Author

request is sent :)

config.yaml Outdated
- name: mdl-js
teams:
mdl-js-maintainers: maintain
- name: hdcx-js
Copy link
Contributor

Choose a reason for hiding this comment

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

hdcx-js -> hDCX-js

case is important

@ryjones
Copy link
Contributor

ryjones commented Aug 12, 2025

@lukasjhan please change hdcx-js to hDCX-js, or tell me to rename the repo to hdcx-js

@lukasjhan
Copy link
Member Author

@lukasjhan please change hdcx-js to hDCX-js, or tell me to rename the repo to hdcx-js

Oh you already changed it :) Thank you. It's all good!

@ryjones
Copy link
Contributor

ryjones commented Aug 12, 2025

@lukasjhan you need to change the YAML above

Signed-off-by: Lukas.J.Han <[email protected]>
@clowarden
Copy link

clowarden bot commented Aug 12, 2025

Validation succeeded

✅ The proposed configuration changes are valid!

Configuration changes

Directory

  • team hdcx-js-maintainers has been added
    • Maintainers
      • lukasjhan
    • Members
      • aceshim
      • yunseorim1116

Github

  • repository hDCX-js has been added (visibility: public)
    • Teams
      • hdcx-js-maintainers: maintain

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

@lukasjhan
Copy link
Member Author

@lukasjhan you need to change the YAML above

Changed it :)

@lukasjhan lukasjhan requested a review from ryjones August 12, 2025 13:53
@ryjones ryjones merged commit 1bd755c into openwallet-foundation-labs:main Aug 12, 2025
3 checks passed
@clowarden
Copy link

clowarden bot commented Aug 12, 2025

Reconciliation completed

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

Changes applied

Github

  • team hdcx-js-maintainers has been added
    • Maintainers
      • lukasjhan
    • Members
      • aceshim
      • yunseorim1116
  • team hdcx-js-maintainers has been added to repository hDCX-js (role: maintain)
  • user lukasjhan is no longer a collaborator of repository hDCX-js

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.

2 participants