Skip to content

Revert "Initial implementation of Command Plugins" #3886

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

Closed
wants to merge 1 commit into from

Conversation

compnerd
Copy link
Member

Reverts #3855

@compnerd
Copy link
Member Author

@swift-ci please test

@compnerd
Copy link
Member Author

@swift-ci please smoke test

@abertelrud
Copy link
Contributor

@compnerd Is there a log where I can see how this failed? It should be possible to fix this forward.

@tomerd
Copy link
Contributor

tomerd commented Nov 30, 2021

@abertelrud @compnerd is this addressed by #3888 and can be close?

@compnerd
Copy link
Member Author

@tomerd - I want to keep this open until we know that it does fix the build, otherwise, I think that we should roll back to get the builds green again - they have been broken for ~12 hours now.

@abertelrud
Copy link
Contributor

@abertelrud @compnerd is this addressed by #3888 and can be close?

In my view we can close this, since I'm confident that we can fix this forward. We are still testing #3888 but I don't see a circumstance in which we'd need to revert this commit.

@compnerd compnerd closed this Nov 30, 2021
@compnerd compnerd deleted the revert-3855-eng/command-plugins branch November 30, 2021 21:30
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