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 LSP-webhare #44

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Add LSP-webhare #44

wants to merge 1 commit into from

Conversation

krisbreuker
Copy link

No description provided.

@rchl
Copy link
Member

rchl commented May 25, 2021

Was support for gitlab fixed in Package Control already? If so I will have to update the review bot.

@krisbreuker
Copy link
Author

It looks like support for gitlab has been merged into Package Control in wbond/package_control#1481.

@rchl
Copy link
Member

rchl commented May 25, 2021

Yes, it was but then someone actually tried it and it didn't actually work - wbond/packagecontrol.io#138

@krisbreuker
Copy link
Author

Ah, I see. I guess I'll leave this PR open until that issue gets resolved?

@TerminalFi
Copy link
Contributor

@krisbreuker @rchl I believe this PR Gitlab PR will resolve this

@krisbreuker
Copy link
Author

As the package control PR has been merged and the package mentioned in the comment above is now available in package control, it seems to me that the Sublime LSP repository should now also be able to accept GitLab packages.

@rchl rchl closed this Aug 16, 2021
@rchl rchl reopened this Aug 16, 2021
@rchl rchl closed this Aug 16, 2021
@rchl rchl reopened this Aug 16, 2021
Copy link

@STReviewBot STReviewBot left a comment

Choose a reason for hiding this comment

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

Automated testing result: ERROR

Repo link: LSP-webhare
Results help

Packages added:
  - LSP-webhare

Processing package "LSP-webhare"
  - ERROR: Invalid "details" value "https://gitlab.com/webhare/lsp/LSP-webhare" for one of the packages.

@rchl
Copy link
Member

rchl commented Aug 16, 2021

Unfortunately your repo is still not supported as it's within a "Group" in which case the URL has different structure.

Both the repository tests (https://github.com/wbond/package_control_channel/blob/a01883ea86a73ce8a61de32e3b86229accd15199/tests/test.py#L185-L202) and especially the Package Control (https://github.com/wbond/package_control/blob/master/package_control/clients/gitlab_client.py) don't support that currently.

@FichteFoll

@FichteFoll
Copy link

Please report this at the PC repo if you haven't already.

@rchl rchl marked this pull request as draft August 26, 2021 09:03
@TerminalFi
Copy link
Contributor

Going to look into this. As I thought we took groups into consideration

@TerminalFi
Copy link
Contributor

Update, I am working to fix Gitlab once again :)

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.

5 participants