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

Package maintainers #332

Closed
brotzeit opened this issue Nov 5, 2019 · 20 comments
Closed

Package maintainers #332

brotzeit opened this issue Nov 5, 2019 · 20 comments

Comments

@brotzeit
Copy link
Member

brotzeit commented Nov 5, 2019

We obviously need active maintainers for this package. Volunteers ? I will try to help.

@sebastiencs
Copy link
Member

Hi @brotzeit

I've been quite busy the last few months and didn't find time to maintain lsp-ui.
I have more time now and plan to commit more on lsp-ui.

I just submitted some changes to emacs in order to fix all (hopefully 🤞) the problems with lsp-ui-sideline.
More changes will follow on emacs to improve lsp-ui.

With that being said, new maintainers are welcome !

cc @yyoncho @MaskRay

@MaskRay
Copy link
Member

MaskRay commented Nov 6, 2019

Nice! I see that @brotzeit has already been added to the lsp-ui team.

@brotzeit
Copy link
Member Author

brotzeit commented Nov 7, 2019

lsp-ui seems to have a large user base. It would be a shame if the package stays unmaintained. @sebastiencs great to hear that you will have time again. I will try to contribute, but I will need some time to get familiar with the code base.

@danielmartin
Copy link
Collaborator

Let me know if I can help.

@brotzeit
Copy link
Member Author

@danielmartin That would be great.

@brotzeit
Copy link
Member Author

@yyoncho @sebastiencs @MaskRay Can somebody please give @danielmartin write access.

@yyoncho
Copy link
Member

yyoncho commented Nov 17, 2019

@yyoncho @sebastiencs @MaskRay Can somebody please give @danielmartin write access.

Done

@jcs090218
Copy link
Member

Are you guys still looking for more maintainers? I'm interested to co-maintain this package.

Let me know if I can help! Thanks!

@brotzeit
Copy link
Member Author

brotzeit commented Sep 5, 2020

Sure. @yyoncho please invite @jcs090218.

@jcs090218
Copy link
Member

Is there anything that may hamper this? 😖 Thanks! 😄

@yyoncho
Copy link
Member

yyoncho commented Sep 6, 2020

Sure. @yyoncho please invite @jcs090218.

Done. I think that you have the right to do that too.

@yyoncho
Copy link
Member

yyoncho commented Sep 6, 2020

@jcs090218

Is there anything that may hamper this? Thanks!

This seems to be the most wanted and the most requested enhancement - #296 .

Also in you are interested in language-specific work you may share the language servers that you use. This is usually the most interesting work since it will directly affect your daily coding activities. And finally, if you want to implement a certain feature/bug don't hesitate to ask here or in gitter chat.

@jcs090218
Copy link
Member

jcs090218 commented Sep 6, 2020

This seems to be the most wanted and the most requested enhancement - #296 .

Let me investigate this feature! I am currently cleaning up lsp projects; meanwhile I am also reading the code base implementation simultaneously while I am doing that. Sorry that I regularly fix code and worry readability before I implement next new feature.

Also in you are interested in language-specific work you may share the language servers that you use. This is usually the most interesting work since it will directly affect your daily coding activities. And finally, if you want to implement a certain feature/bug don't hesitate to ask here or in gitter chat.

No problem! I code a lot of languages TBH. I usually change my daily programming language base on different circumstances. The recent programming languages I have used are javascript, typescript and csharp. But I don't often bother lsp-mode if it doesn't get things right. Unless it really poorly executed. 😖


@yyoncho Anyway, Would you mine if I joined the organization? I might want co-maintain multiple lsp packages, and I think is easier for you manage me as a team using GitHub organization's feature.

These are packages I am interested in.

Interested but probably less activity because there are multiple very active maintainers (I assumed)

  • lsp-mode - of course
  • dap-mode
  • lsp-java - I love java but I haven't code in java for a while.

Interested and I am more likely to be active.

  • lsp-ui - This is probably the most important package for me now.
  • lsp-ivy - Yes, I use ivy. :P

Other packages that I have experienced with and able to help

  • helm-lsp - I am able to help.
  • lsp-focus
  • lsp-origami

Let me know what you think! :)

@yyoncho
Copy link
Member

yyoncho commented Sep 6, 2020

I added you to the organization. Please, before starting to work on a reported issue, assign it to you or drop a note to avoid collisions.

Interested but probably less activity because there are multiple very active maintainers

Still, there are a lot of open tasks there.

@jcs090218
Copy link
Member

I added you to the organization. Please, before starting to work on a reported issue, assign it to you or drop a note to avoid collisions.

Thanks! I appreciated! Will do!

Still, there are a lot of open tasks there.

Got you! If any package needs help, let me know!

@yyoncho
Copy link
Member

yyoncho commented Sep 6, 2020

Got you! If any package needs help, let me know!

It is not up to me - if you like a task and you have free time go ahead and start working on it. I can only advise what in my opinion is with the highest priority.

@anonimitoraf
Copy link
Contributor

Hey, interested in helping out too

@brotzeit
Copy link
Member Author

brotzeit commented Dec 2, 2020

@anonimitoraf do you know how to write elisp ? Usually we add contributors to the project when they've added a few commits. You can always open pull requests when you want to add some code. Get familiar with the lsp-ui codebase and the issue tracker. Unfortunately we currently don't have an active maintainer that knows enough about the code base to review larger pull requests properly. I'm always hesitant to merge pull requests since we often had the situation that we fixed one bug and created a new one. Writing tests for lsp-ui is also very difficult(often simply not possible) and currently we have no idea if our code is actually working.

@anonimitoraf
Copy link
Contributor

I've started learning

Usually we add contributors to the project when they've added a few commits. You can always open pull requests when you want to add some code

Right, Makes sense

Writing tests for lsp-ui is also very difficult(often simply not possible)

Why is this the case?

@brotzeit
Copy link
Member Author

brotzeit commented Dec 3, 2020

For example you can't make additional frames when running tests for emacs so you can't add tests for ui-doc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants