Skip to content
This repository has been archived by the owner on Nov 12, 2017. It is now read-only.

moving to REditorSupport #25

Closed
randy3k opened this issue Sep 12, 2017 · 14 comments
Closed

moving to REditorSupport #25

randy3k opened this issue Sep 12, 2017 · 14 comments

Comments

@randy3k
Copy link

randy3k commented Sep 12, 2017

Hi @lee-dohm

It is pity that you are no longer supporting this. @Ikuyadeu (author of VSCode R package) and I (author of Sublime Text R package) have recently created an organization: https://github.com/REditorSupport
And we have an idea to transfer our packages, R-Box and vscode-R to the organization and use Language Server Protocol to provide the language support on those platforms. The transfer of R-Box will happen when the language server for R is ready.

I am just wondering if you will be also interested in transferring this to the organization. Although I cannot guarantee that much, with the language server that we are developing and atom-languageclient. At least there will be some minimum support for the R language in Atom, better than being stale in this repo.

@randy3k
Copy link
Author

randy3k commented Sep 12, 2017

I have just invited you to the organisation just in case you want to transfer the package to there.

@lee-dohm
Copy link
Owner

Thank you very much for the offer.

Whether this repo lives here or in @REditorSupport, I simply don't have the time or expertise to maintain it anymore. It is better if someone else who is actively using R takes up the effort. If you or @Ikuyadeu are interested in taking this on, please feel free to fork the code from here as a starting point.

@randy3k
Copy link
Author

randy3k commented Sep 12, 2017

I am inviting you to join because I also want to inherit the package name on atom.io: https://atom.io/packages/language-r
If we fork your repo, we probably need to use a different name and it is less desirable unless there is a way to overtake a deprecated package on atom.io.

@lee-dohm
Copy link
Owner

There isn't a way to forcibly take over someone else's package. A package can be transferred by transferring the underlying repository. I could do that, but it doesn't sound like anyone is really signing up to take on maintenance of the package?

@randy3k
Copy link
Author

randy3k commented Sep 12, 2017

I have a plan to at least implement the R language server on atom. It doesn't seem to be very hard
to use language client once the R language server is ready to ship.

If you and (other phantom collaborators) don't mind, I will be happy to take over it at REditorSupport. Given that it is very likely that @Ikuyadeu and I will continuously support vscode and sublime text (plus the language server). Transferring it to an umbrella organization may be a better choice than waiting for somebody to fork the repo.

@randy3k
Copy link
Author

randy3k commented Sep 12, 2017

By the way, if you don't wan to transfer it now. We can wait until @Ikuyadeu and I have done more work at REditorSupport. And I will come back to you a bit later when I am ready to implement the language server on Atom.

@lee-dohm
Copy link
Owner

We can wait until @Ikuyadeu and I have done more work at REditorSupport.

Thanks. Given my experience with adding collaborators here, I'd rather base my decision on concrete progress than people's stated plans 😀

@randy3k
Copy link
Author

randy3k commented Sep 12, 2017

Totally understood.

@guillaumechaumet
Copy link
Contributor

Nice project @randy3k. I made an attempt to produce a autocomplete package for R. Perhaps we could collaborate together to improve communication between R and atom-io.

@randy3k
Copy link
Author

randy3k commented Sep 13, 2017

For the atom R support, my plan is to move it to language server protocol so it will be easier to be maintained across different editors. Feel free to implement languageserver via atom language client.

I guess if we could show our commitment, @lee-dohm will be more confident to transfer language-r to REditorSupport.

@randy3k
Copy link
Author

randy3k commented Sep 13, 2017

Though, I should have added that the language server is not ready yet. Only code diagnostic is implemented.

@kylebarron
Copy link

Hi @lee-dohm and @randy3k ,

Over the summer I saw the calls for a volunteer maintainer, and would have replied but wanted to have some proof of work first. I completely rewrote the Stata language package (another statistical package heavily used by economists) and the maintainers of the previous package decided to make mine the main one in Atom.

I'm 23 commits into an update of language-r. The main areas for improvement I've been working on are better highlighting of function calls and regular expression highlighting. I had hoped to ask to be a volunteer maintainer upon finishing my updates and submitting a large PR. I find less utility in making a language-r-plus package in Atom that's hard to find and has relatively small probability of getting off the ground, user-base wise.

@randy3k keep me in the loop if you fork the language package to REditorSupport, and I'd be happy to collaborate on the fork there, if desired.

@randy3k
Copy link
Author

randy3k commented Sep 14, 2017

@kylebarron
I have pushed a temporary repo for atom-language-r and invited you to REditorSupport. Please feel free to push to it.

PS: I intentionally did not "fork" this repo so @lee-dohm can transfer this repo to REditorSupport later.

@randy3k
Copy link
Author

randy3k commented Sep 14, 2017

@guillaumechaumet

I have also invited you to the project.

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

No branches or pull requests

4 participants