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

feat: add new group for dart #404

Merged
merged 4 commits into from
Nov 21, 2024
Merged

feat: add new group for dart #404

merged 4 commits into from
Nov 21, 2024

Conversation

thomaspoignant
Copy link
Member

This PR

Following our community meeting here is the new group to prepare contributors for dart sdk server.

Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>
@@ -0,0 +1,8 @@
repos:
- dart-sdk-server
Copy link
Member Author

Choose a reason for hiding this comment

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

This repo name is a proposal.

Copy link
Member

Choose a reason for hiding this comment

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

How about dart-server-sdk?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes sounds even better, I will rename it 👌

thomaspoignant and others added 3 commits November 21, 2024 17:08
Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>
Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>
@thomaspoignant
Copy link
Member Author

I am not sure why the CI is failing with the error

Configuration failed: failed to configure open-feature teams: failed to update sdk-dart-approvers members: failed to list sdk-dart-approvers(sdk-dart-approvers) members: return code not 2XX: 404 Not Found

@aepfli any idea on what can go wrong?

@toddbaert
Copy link
Member

toddbaert commented Nov 21, 2024

I am not sure why the CI is failing with the error

Configuration failed: failed to configure open-feature teams: failed to update sdk-dart-approvers members: failed to list sdk-dart-approvers(sdk-dart-approvers) members: return code not 2XX: 404 Not Found

@aepfli any idea on what can go wrong?

This happens just on PRs that add teams because the rely on the team to exist to add people to it - it's expected but we add teams so rarely we have never addressed it. When you merge this it should work fine.

I (and I think you?) can force merge it anyway and it will work on main.

@thomaspoignant
Copy link
Member Author

@toddbaert alright got it, I will merge it.

@thomaspoignant thomaspoignant merged commit 8396291 into main Nov 21, 2024
1 of 2 checks passed
@thomaspoignant thomaspoignant deleted the setup-dart branch November 21, 2024 16:52
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.

3 participants