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 vscode GitHub Pull Requests plugin #363

Merged
merged 3 commits into from
Jan 11, 2021
Merged

Add vscode GitHub Pull Requests plugin #363

merged 3 commits into from
Jan 11, 2021

Conversation

vinokurig
Copy link
Contributor

What does this PR do?

Add vscode GitHub Pull Requests plugin to the plugins list page.

What issues does this PR fix or reference?

https://issues.redhat.com/browse/CRW-1226

Release Notes

Docs PR (if applicable)

Signed-off-by: Igor Vinokur <ivinokur@redhat.com>
@nickboldt nickboldt changed the base branch from crw-2.5-rhel-8 to crw-2.6-rhel-8 November 3, 2020 18:34
Copy link
Member

@nickboldt nickboldt left a comment

Choose a reason for hiding this comment

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

-1, changes requested inline

@nickboldt
Copy link
Member

Assuming that this new plugin will need to be documented as it'll improve git-related workflows in Che/CRW, please also link to

Docs PR (if applicable)

or open a related RHDEVDOCS issue to track the work separately.

@ericwill
Copy link
Contributor

Building this extension on the jenkins pipeline fails (of course). This PR may have to wait until I get the Dockerfile based build system up and running (hopefully before Christmas).

@MichalMaler
Copy link

@ericwill Hello Eric! Let me know when I can put this back in CRW docs. I think it will be in January, right? Cheers

@ericwill
Copy link
Contributor

@ericwill Hello Eric! Let me know when I can put this back in CRW docs. I think it will be in January, right? Cheers

Yes, early January sounds good.

@nickboldt
Copy link
Member

@vinokurig can you apply requested changes to this PR so we can merge it into 2.6?

@ericwill
Copy link
Contributor

ericwill commented Dec 3, 2020

We still need to build the GitHub PR plugin ourselves, in https://github.com/redhat-developer/codeready-workspaces-vscode-extensions

I started this work but it's still pending.

@vinokurig
Copy link
Contributor Author

vinokurig commented Dec 18, 2020

I managed to build the plugin in my fork: https://github.com/vinokurig/codeready-workspaces-vscode-extensions/releases/tag/v7076c43 by updating the plugin version to 0.21.4. The build of the current 0.20.0 version fails, probably the author did some patches to publish it. The problem is that the 0.21.4 version doesn't work in che-theia due to missing API. I've filed an issue to update the plugin in Che: eclipse-che/che#18661

@ericwill
Copy link
Contributor

ericwill commented Jan 5, 2021

I managed to build the plugin in my fork: https://github.com/vinokurig/codeready-workspaces-vscode-extensions/releases/tag/v7076c43 by updating the plugin version to 0.21.4. The build of the current 0.20.0 version fails, probably the author did some patches to publish it. The problem is that the 0.21.4 version doesn't work in che-theia due to missing API. I've filed an issue to update the plugin in Che: eclipse/che#18661

Florent was able to build the extension, here is the link: https://github.com/redhat-developer/codeready-workspaces-vscode-extensions/releases/download/vdb8b64f/vscode-pull-request-github-0.20.0.vsix

Let's include 0.20.0 in CRW 2.6 and we can explore updating to 0.21.4 later on.

@nickboldt
Copy link
Member

OK, I'm good to merge this once someone fixes the date '2020-10-8' ... not only is the day only a single digit, it's 3 months ago. Please bump that to a date in Jan 2021 for accuracy.

@MichalMaler
Copy link

Great stuff and nice progress! Let me know when I can return the docs about this plugin back to CRW. BTW, do we need to update the Che-relative content?

https://www.eclipse.org/che/docs/che-7/end-user-guide/version-control/#accessing-a-git-repository-via-ssh_version-control

@ericwill
Copy link
Contributor

ericwill commented Jan 6, 2021

Great stuff and nice progress! Let me know when I can return the docs about this plugin back to CRW. BTW, do we need to update the Che-relative content?

https://www.eclipse.org/che/docs/che-7/end-user-guide/version-control/#accessing-a-git-repository-via-ssh_version-control

When this PR is merged then you can restore the docs. As for the content, I'll have Igor verify the content with you when he gets back from PTO next week.

@MichalMaler
Copy link

Ok, great. Thx Eric

@ericwill
Copy link
Contributor

ericwill commented Jan 6, 2021

OK, I'm good to merge this once someone fixes the date '2020-10-8' ... not only is the day only a single digit, it's 3 months ago. Please bump that to a date in Jan 2021 for accuracy.

I've pushed a new version to fix the date and the vsix reference, I think we can merge this one now

@vinokurig
Copy link
Contributor Author

I've opened a doc's PR that enables the GitHub documentation for CRW: eclipse-che/che-docs#1784

@vinokurig
Copy link
Contributor Author

@nickboldt Could you please review it again?

@nickboldt nickboldt merged commit b976895 into redhat-developer:crw-2.6-rhel-8 Jan 11, 2021
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.

4 participants