-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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 Simplenote package #8951
Add Simplenote package #8951
Conversation
I wonder why that isn't a proper fork with all the history preserved. Could you make it a proper fork? And then we may consider replacing the old plugin with your new update. You don't claim any breaking changes either, just bug fixes and the general shift to python38 and ST4. Have you actually reached out to the old maintainer? |
I have been using the original plugin for a while, but it seems that the author is no longer updating it. I checked the code and found that there were many bugs in the new version of Sublime Text, and the code structure was quite cumbersome. Therefore, I rewrote the plugin, only borrowing the ideas, settings, and hotkey bindings. In fact, the code refactoring is about 70% complete. |
In the future, I plan to add some new features to the plugin:
|
I like it. This should still be a takeover because we don't need both packages, do we? E.g. we could pin Sublime Text 3 users to an old tag. For this to work, we would have needed the original commits, even if they would point to a different root. But you did not import/fork the old tags either. I probably need to clone your repo because I don't know if the GitHub webpage is just too limited to show the whole repo story. Let's also wait what @braver says. IMO it should be a takeover with the minor package rename, and with a pinned old version that refers the last commit/tag of the original author. I really would have liked if @RedAtman had preserved the history of the code/project. |
I re-forked the repository, preserving the existing tags, and created new tags for my new version. |
Ah okay, you're moving fast here. Something is wrong with settings management as you must use Thus, the following will not work https://github.com/RedAtman/simplenote/blob/bfb861bb5d138019df832dcbadf0a9ed05e1ea1a/simplenote.py#L31-L41
|
I have refactored the logic for the settings. Please check if it meets our expectations. |
The settings handling is looking good. But keep in mind that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Automated testing result: WARNING
Repo link: Simplenote
Results help
Packages added:
- Simplenote
Processing package "Simplenote"
- WARNING: The package does not contain a top-level LICENSE file. A license helps users to contribute to the package.
Okay, your suggestion was good. I have completed it and created a new tag. |
|
I'm all done. You are very patient. |
What else needs to do? |
Some final tweaks to be made:
Looks like this is an improved version of the original. Unless @sickmartian opposes, we should replace the existing package so that existing users also finally get an update. Or maybe he could transfer the repo or otherwise let you manage the existing package. Let's give @sickmartian a few weeks to respond, but otherwise a takeover seems like the best way forward. |
Thanks @braver for the tag, I was not aware of this fork until your message. I would actually oppose a take over, while yes the original is discontinued and won't be updated anymore, any user that installed it might've gone through the repo and seen the docs and getting updates from a different repo sounds disingenuous and a recipe for supply chain attacks. I think the package should stand on it's own. If you don't want duplicate packages you can remove mine, it's shitty for any remaining users that they will actually need to find and find the new plugin and install it, but I think them actually opting-in into the new source of code that will be executed into their machines and a new maintainer is the proper thing to do. I won't point users to a new code/repo I haven't seen and a maintainer I don't know, same reason I won't transfer my repo or make them maintainer, they never reached out and did it on their own, so I think it should properly stand on it's own. |
Well I guess that’s a no 😉 While we prefer packages to be kept “alive” through contribution, forking, or whatever means are practical for that particular case, it’s not a requirement. You own your package and decide what happens with it. So @RedAtman please just report back on that latest feedback and we’ll continue working towards shipping your addition to package control. |
Thank you for your suggestions, I have now completed everything. and created a new release. |
Thank you all for taking this matter seriously. I am new to the open-source community, and my previous actions might have been a bit hasty. If @sickmartian still have time to update the code together., I would be honored to contribute to the original code and merge my new code into the original repository. If the answer is no, I am also happy to offer my new code as a separate package. In the future, I might add some new features, such as replacing the current ReSTful API with Websocket, using sqlite as the database, or adding APIs for other notebook providers. I like the simplicity of Simplenote and its extensible advantages in text display. However, the official client does not support fonts and styles very well. Therefore, I want to create a new client using Sublime Text's beautiful styles and rich syntax support. In any case, I am willing to fully respect @sickmartian 's opinion and am grateful for the foundation you provided, which allowed me to quickly complete the current code, adapted for the latest version of Sublime Text. |
Perfect, thanks for understanding 👍
You didn't do anything hasty on my view, you took over a dead project and made it alive again, it's great and what should happen. I'm just not comfortable giving control over the repo to someone I don't know in case there are active users using the plugin, feels like a responsibility I don't want. I also don't want to continue development or to audit new code for this. So, I think you going with your own repo and not merging into mine is the way to go. Bests of luck on the new project 👍 |
Alright, let’s go. If you do end up collaborating or whatever we can always merge the packages. For now, we’ll ship this. |
My package is Simplenote
My package is similar to Quick Simplenote, However it should still be added because Sometimes it doesn't work properly in Sublime Text 3/4.
Run the ChannelRepositoryTools: Test Default Channel command:
...............................................................................................................
Ran 9727 tests in 0.182s
OK