Skip to content

docs: change the clone url #307

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

Merged
merged 4 commits into from
Sep 3, 2024
Merged

Conversation

noam-honig
Copy link
Contributor

the [email protected] didn't work on my machine

Also - when trying to set this up, it required the specific [email protected] for some reason - so I've added that to the document

 the [email protected] didn't work on my machine

Also - when trying to set this up, it required the specific [email protected] for some reason - so I've added that to the document
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@noam-honig noam-honig changed the title Change the clone url docs:Change the clone url Sep 3, 2024
@noam-honig noam-honig changed the title docs:Change the clone url docs: Change the clone url Sep 3, 2024
@noam-honig noam-honig changed the title docs: Change the clone url Docs: Change the clone url Sep 3, 2024
@noam-honig noam-honig changed the title Docs: Change the clone url docs: change the clone url Sep 3, 2024
@noam-honig
Copy link
Contributor Author

@AriPerkkio Added your change and also added where one can look for the specific verison - do you think its required?

@AriPerkkio
Copy link
Member

Added your change and also added where one can look for the specific verison - do you think its required?

Corepack can automatically detect and use the correct version of defined package manager, so there's no need for developers to check what version is there. I think it's better to delete that part from the docs.

@AriPerkkio AriPerkkio merged commit 2e259fa into stackblitz:main Sep 3, 2024
10 checks passed
@noam-honig
Copy link
Contributor Author

@AriPerkkio Thanks - sorry, I wasn't available for a few hours to remove the line you wanted

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

Successfully merging this pull request may close these issues.

2 participants