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

Use remote file handle for git init step #5118

Merged
merged 1 commit into from
Oct 22, 2024
Merged

Conversation

timtmok
Copy link
Contributor

@timtmok timtmok commented Oct 22, 2024

Address #5089

Added a file scheme to the project config that is stored. This is used to recreate the URI with the correct scheme.

I think the git init command is fine since it runs on the server. The error was occurring when trying to create the .gitignore and README.md since it calls the file service.

QA Notes

git init can use local file scheme since it runs on the extension host
creating .gitignore and readme need the correct file scheme to pass to file service
Copy link
Member

@sharon-wang sharon-wang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 👍

Looks good on Positron Server Web on Mac, for Python, R and Jupyter projects. README, gitignore and .git were all created as expected.

@timtmok timtmok merged commit be7c0d9 into main Oct 22, 2024
3 checks passed
@timtmok timtmok deleted the bugfix/new-project-git-init branch October 22, 2024 20:52
@github-actions github-actions bot locked and limited conversation to collaborators Oct 22, 2024
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