Skip to content
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

Allow to edit files and then create a pull request #21

Closed
justanotheranonymoususer opened this issue Feb 9, 2021 · 15 comments · Fixed by #214
Closed

Allow to edit files and then create a pull request #21

justanotheranonymoususer opened this issue Feb 9, 2021 · 15 comments · Fixed by #214
Labels
enhancement New feature or request

Comments

@justanotheranonymoususer
Copy link
Contributor

Or at least export a patch file or something. That would be really cool 😀
Awesome project!

@ThiefMaster
Copy link

I don't care so much about the PR, but being able to edit a file just to copy slightly-changed code to paste somewhere would be great.

@conwnet
Copy link
Owner

conwnet commented Feb 9, 2021

I agree with this, and It is really possible to change file content and create an commit online through GitHub API Docs, I can work for it in nearly future.

@ThiefMaster as for simple edit and paste code, you can choose "File -> New File" in activity bar to create an empty file and edit it, may be we can offer an quick way (may be a button or a command) to do it.

@ThiefMaster
Copy link

I meant an existing file in the repo - for example, when you are discussing something in an issue and want to paste a snipped from the repo with a some minor modifications.

@ritchxu
Copy link

ritchxu commented Feb 9, 2021

GitHub has its own Codespaces product coming for that and more https://github.com/features/codespaces

@notpushkin
Copy link

@ritchxu There is also gitpod.io and some other offerings that allow this. I believe github1s is the “light” way to quickly look at (and probably edit) the repo (zero startup time, but no features requiring server-side support, like terminal or external language servers).

@xcv58
Copy link
Collaborator

xcv58 commented Feb 10, 2021

I think this feature is doable, but we have to finish #12 and make some refactors to support this.

@mainrs
Copy link

mainrs commented Feb 10, 2021

I wanted to chime in and give another point of view. I kind of disagree with the PR feature. At least I wouldn't put my focus a lot of focus into it (i.e. high priority feature). The great thing about github1s is that it spins up way faster than codespaces (cold and hot). It's great for poking around in a code base or when trying to see how someone did something without having to clone it.

As soon as it comes to editing though, I personally think that spinning up a codespace is superior. It supports persistence once created, your dotfiles are automatically copied over and it is tighter integrated into GitHub. It even allows for extensions installation IIRC .

I personally think that maybe an edit in codespace button would be great if the repository has a devcontainer.json configured. I am not that familiar with Codespace APIs but maybe it's even possible to automatically create a PR with some default devcontainer.json from within github1s, as some kind of bridging between the two. This would then allow people to introduce maintainers to Codespaces :)

Looking forward to hear other opinions and thoughts!

@justanotheranonymoususer
Copy link
Contributor Author

@mainrs
Copy link

mainrs commented Feb 10, 2021

I just assumed that because it's VSCode under the hook it would be the fully fledged experience :) A really simple editing functionality would probably be beneficial indeed!

@xcv58
Copy link
Collaborator

xcv58 commented Feb 10, 2021

I think simple editing + downloadable patch is good enough.

@braebo
Copy link

braebo commented Feb 11, 2021

I’ve been replacing my GitHub graphql api powered REPL with isomorphic git running in web workers. Could that be good for this use case perhaps?

@xcv58
Copy link
Collaborator

xcv58 commented Feb 11, 2021

I’ve been replacing my GitHub graphql api powered REPL with isomorphic git running in web workers. Could that be good for this use case perhaps?

@fractalhq Could you please share more context? Any repo/demo will be appreciated.

@xcv58 xcv58 added the enhancement New feature or request label Feb 11, 2021
@Felx-B
Copy link

Felx-B commented Feb 18, 2021

At some point, I tried to do something like this, which cloned an entire repository in IndexedDB using isomorphic git.
https://github.com/Felx-B/vscode-gitfs/blob/fbo/init/gitfs/src/extension.ts

But to use it in VSCode UI, you need to reimplements git features in VSCode through SourceControl

@justanotheranonymoususer
Copy link
Contributor Author

It's not quite what I meant, Gitpod doesn't take 1s to load :(

@conwnet
Copy link
Owner

conwnet commented Mar 12, 2021

It's not quite what I meant, Gitpod doesn't take 1s to load :(

I know, it is for someone who need full ide abilities now, we will make continuous improvement in this feature. ( quick edit in 1s)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants