-
Notifications
You must be signed in to change notification settings - Fork 105
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
decaffeinate and update deps #464
Conversation
I am planning on updating the other dependencies while I'm at it, I figured I would open this PR for discussion if there is anything specific I should use. (like eslint config) |
@atom-community if you need a new maintainer for this repo (and/or other repos) I would love to help out. |
@UziTech If there is no movement here you should release your fork as a new package 👍 |
@dirk-thomas are you still maintaining this repo? |
No, I have moved away from Atom quite some time ago. |
I suggest to rather poke the current owner of the repo to consider giving permissions to a new maintainer rather than forking. |
@dirk-thomas do you know who the current owner is? |
@UziTech I just sent you an invite to join @atom-community/sync-settings-maintainers. Thanks for offering to help, and let me know if you need anything else! ❤️ |
@smashwilson Thanks. Is there any guidelines on maintaining this repo (e.g. use certain eslint config, etc.) and am I able to publish releases to atom.io? |
I use |
I don't know of any. But @simurai also literally just added me to this org last night, so I wouldn't consider myself an authority 😅
Looks like it, so hooray! |
As far as I know anyone with write access to a repository can publish versions to atom.io. |
Decaffeinate and update dependencies
fixes #463
I added
eslint-config-standard
for the eslint config. The code isn't very pretty from being decaffeinated but it works.All dependencies are updated.
I updated the
package-manager
script as well fromsettings-view
Removed dependencies
emissary
andq
Tests all pass locally. It looks like travis doesn't have a valid PAT so api calls are rate limited.