Skip to content
This repository was archived by the owner on Nov 2, 2020. It is now read-only.

Incomplete PR to support new RF and SeleniumLiumrary #230

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

aaltat
Copy link
Collaborator

@aaltat aaltat commented Feb 16, 2019

Should not be pushed to master. Only some experimental code how to fix plugin to support RF and SL.

@esaruoho
Copy link

Hi, what should be done about this PR, please?

@aaltat
Copy link
Collaborator Author

aaltat commented Dec 1, 2019

Well work from someone. Also there is #231 which takes the work further. I think in this point I would stop supporting Python 2 and support only Python 3.

In reality this project would need new maintainer. Robot Framework has syntax has changed quite a lot and most likely will change more in the future releases. Also the new parser in RF 3.2 offers new possibilities and work done for supporting language server protocol, https://langserver.org/ is also tempting. Would you happen to know if someone is interested?

@esaruoho
Copy link

esaruoho commented Dec 1, 2019

@aaltat While I do understand issues, branches and PRs and can somewhat comprehend diff + code review concepts and testing a branch out to see if it works, I don't have enough of an understanding of Sublime Text 3, Robot Framework or the languages these are running in, to be an actual, useful maintainer.

@aaltat
Copy link
Collaborator Author

aaltat commented Dec 1, 2019

Well, I could say: When I started my maintenance sift for the plugin, I don't have either. I was driven by a need and rest can be learned along the way.

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

Successfully merging this pull request may close these issues.

2 participants