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

Add TSP project #8

Merged
merged 2 commits into from
Jun 3, 2024
Merged

Conversation

tkuhrt
Copy link
Contributor

@tkuhrt tkuhrt commented May 30, 2024

No description provided.

@tkuhrt tkuhrt requested a review from ryjones as a code owner May 30, 2024 14:20
Copy link

clowarden bot commented May 30, 2024

Validation failed

‼️ Some errors were found validating the configuration files


Errors:

  • invalid github service configuration
    • team[tsp-maintainers]: wenjing must be an organization member to be a maintainer

For more details about the configuration files format please see the documentation.

🔺 These errors must be addressed before this PR can be merged 🔺

config.yaml Show resolved Hide resolved
Copy link
Contributor

@ryjones ryjones left a comment

Choose a reason for hiding this comment

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

I mentioned how to fix the error

@ryjones
Copy link
Contributor

ryjones commented May 30, 2024

@tkuhrt I think you need to do a rebase to trigger CLOWarden again.
git commit --amend --no-edit
git push --force-with-lease ...

@ryjones ryjones self-requested a review May 30, 2024 16:07
Signed-off-by: Tracy Kuhrt <tracy.a.kuhrt@accenture.com>
Copy link

clowarden bot commented May 30, 2024

Validation failed

‼️ Some errors were found validating the configuration files


Errors:

  • invalid github service configuration
    • team[tsp-maintainers]: wenjing must be an organization member to be a maintainer

For more details about the configuration files format please see the documentation.

🔺 These errors must be addressed before this PR can be merged 🔺

Signed-off-by: Tracy Kuhrt <tracy.a.kuhrt@accenture.com>
Copy link

clowarden bot commented Jun 3, 2024

Validation succeeded

✅ The proposed configuration changes are valid!

Configuration changes

Directory

  • team credhub-maintainers has been removed
  • team tsp-maintainers has been added
    • Maintainers
      • tkuhrt
    • Members
      • marlonbaeten
      • tweedegolf-marc
      • wenjing

Github

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

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

@ryjones ryjones merged commit feb5bd7 into openwallet-foundation-labs:main Jun 3, 2024
2 checks passed
Copy link

clowarden bot commented Jun 3, 2024

Reconciliation completed

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

Changes applied

Github

  • team tsp-maintainers has been added
    • Maintainers
      • tkuhrt
    • Members
      • marlonbaeten
      • tweedegolf-marc
      • wenjing
  • repository tsp has been added (visibility: public)
    • Teams
      • tsp-maintainers: maintain

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.

None yet

2 participants