-
-
Notifications
You must be signed in to change notification settings - Fork 391
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
Suggestion: Allow upload to repository #413
Comments
Thanks, will look into this later on. |
Also like this idea. |
I thought gists were repos. Edit: I commented on this issue because I think it could be useful to merge when downloading instead of just clobbering the local configuration. Is this feature on the roadmap for this project? Would you consider a pull request? |
Yes, but not in short time. Will work on it later on.
Definitely, I will more happy to accept PR. But anyone wants to send PR it would be great to discuss as its currently tightly coupled with GIST logics. |
Following are the feature i see for git sync.
|
@BoostIO funded this issue with $200. Visit this issue on Issuehunt |
@shanalikhan funded this issue with $230. Visit this issue on Issuehunt |
This feature is funded, anyone is welcome to work and get the money 😄 |
@mortezaPRK has started working. Visit this issue on Issuehunt |
@shanalikhan funded this issue with $130. Visit this issue on Issuehunt |
@mortezaPRK |
Thanks. I already do about 80% of the work. There are some bugs. It should be done by the end of next week :)
Sent from ProtonMail mobile
…-------- Original Message --------
On Oct 11, 2018, 19:08, Shan Khan wrote:
***@***.***(https://github.com/mortezaPRK)
any update on this ? i can assist you if need any help
—
You are receiving this because you were mentioned.
Reply to this email directly, [view it on GitHub](#413 (comment)), or [mute the thread](https://github.com/notifications/unsubscribe-auth/AIyzR8_AErx91qX3JGryZ0s7Vr80FNkkks5uj2XygaJpZM4P1ktH).
|
Thats great, make sure you write the test cases. It would be great you have send me PR with work in progress so i can see and evaluate it mean while you are doing else once all work is completed it would be difficult to change at end if we found any. |
Sure, I'll send a PR tomorrow.
Sent from ProtonMail mobile
…-------- Original Message --------
On Oct 11, 2018, 20:23, Shan Khan wrote:
Thats great, make sure you write the test cases.
We need to decouple modules in order to implement repo sync rather then writing complete other logic for it.
It would be great you have send me PR with work in progress so i can see and evaluate it mean while you are doing else once all work is completed it would be difficult to change at end.
—
You are receiving this because you were mentioned.
Reply to this email directly, [view it on GitHub](#413 (comment)), or [mute the thread](https://github.com/notifications/unsubscribe-auth/AIyzRw41CgK6zSDzwPPhFI1hdrPggEhFks5uj3eMgaJpZM4P1ktH).
|
@mortezaPRK its been almost a month. It would be great if you can send work in progress PR so i can evaluate at least. |
If i don't get the PR in a week. I will have to assign the paid issue to someone else. |
I might be interested to looking into this |
@OrangeDrangon Anyone feel free to start working on it, this issue or any other. Currently the bounties for 4 issues are above 500$. See the article here. |
Is someone working on it? I can finish first version tomorrow. I have support for detection of gist vs repo url. For now only github, but gitlab API is not so hard to integrate. |
@wasikuss no you can start working on it.
How do you intend to do it, Have a look on steps mentioned in first post - We need to just create git init and stage entire user folder. |
|
We would want to |
The main goal for repository is to create Git Repo, not tightly coupled with github. If you look on the steps, it main focus to create git repository and do pull / push to that repo and then provide some interface options to connect to github. |
So sync with local repo only wraps credentials and @shanalikhan ok, using any client/server convince me to use local repo. but I see upload/download actions acts only as alias for regular git commands and it's only for convenience ( to not enter manually to local repo )? |
I saw similar behavior while using gists (manual change + download). |
Issue is available for bounty worth $200. |
If nobody’s working on this, I’ll start on it when I get home in a few hours. |
Is this still relevant. I can do this! |
To implement this feature, we need to Refactor the complete extension. @arnohovhannisyan and other users have attempted the refactoring thing before. You can use this PR for reference. |
Hi! I used this plugin previously, and I wondered if you still need help with this, or if someone is working on it already! I'd love helping out! 😄 |
OPEN FOR PR! |
This feature would be awesome. Our development environment doesn't have any Internet connection and we have a self-hosted BitBucket server without any snippet add-on. |
I've made a trick workaround to get this, using a python script to sync extensions |
FWIW, your gist is a fully-fledged repository. git clone https://gist.github.com/kiprasmel/de9160a0602463fb752f2d84d7aa4fd8 and from here, you can do all your standard git stuff - amend, reset, force push, etc. e.g. if i sometimes override between syncs, i cd into the repo, pull, reset, and undo the deletions, then commit & push to the gist, and then sync the gist back into vscode. cheers. |
By @shanalikhan
Before starting this feature, Settings Sync required code requires refactoring. You can take #937 as a baseline for refactoring to start working.
Following are the feature i see for git sync.
Originally by @smbkr
It would be useful to be able to upload the settings to a repo instead of a gist, the way that JetBrains IDEs sync, so that the changes could be versioned and rolled back as desired.
IssueHunt Summary
Backers (Total: $200.00)
Become a backer now!
Or submit a pull request to get the deposits!
Tips
IssueHunt has been backed by the following sponsors. Become a sponsor
The text was updated successfully, but these errors were encountered: