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

Added configurable Gitlab domain. #134

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ghost
Copy link

@ghost ghost commented Aug 9, 2021

This (partially) closes issues #44 . I have only a Gitlab custom domain to test with, so that's why I cannot provide a similar change for the Github part.

@gir-bot gir-bot added S: needs-review Needs to be reviewed and/or approved. C: markdown-preview Main code. C: settings Sublime settins files. C: source Related to source code. labels Aug 9, 2021
@facelessuser
Copy link
Owner

@ba-tno thanks for this pull! I'm wondering if it would be useful to make the setting a list and prompt the user for the domain if the setting provides more than one. There may be reasons to use one domain for certain files and another for other files.

This is more a question as I'm just thinking about usefulness.

@ghost
Copy link
Author

ghost commented Aug 9, 2021

Sorry, I see now that I botched it by calling self in the class variable... I'll have to take a look again anyways, so I'll see if I can get a list working as well (something similar to the enabled_parsers setting).

@facelessuser
Copy link
Owner

As things have gone silent on this, I may just end up coming back and refactoring this to address related issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: markdown-preview Main code. C: settings Sublime settins files. C: source Related to source code. S: needs-review Needs to be reviewed and/or approved.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants