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

Improve setup doc #67

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Improve setup doc #67

wants to merge 5 commits into from

Conversation

defnax
Copy link

@defnax defnax commented Jan 27, 2024

No description provided.

@ayushkoli772
Copy link

Im not sure but I think for flutter commands to work, git is one of the requirements so i think along with github desktop, git CLI should also be installed. It wont affect anything even if we install git but maybe it can if its not installed.

image

@defnax
Copy link
Author

defnax commented Jan 27, 2024

ok i will soon change it, yes seems i has installed git lots of years ago i forget it was importent

@defnax
Copy link
Author

defnax commented Jan 27, 2024

but flutterroot and androidroot not realy needed or ,for me not needed

@ayushkoli772
Copy link

but flutterroot and androidroot not realy needed or ,for me not needed

Yes, as most probably everyone use their own path but I think it might be helpful just for reference

@defnax
Copy link
Author

defnax commented Jan 27, 2024

hi check again, i think now the git installer does install automaticly add the env variables?
i has updated my git to latet then the variable has changed automaticly.
after i checked the env has changed i doesnt did anything in env for git

@ayushkoli772
Copy link

hi check again, i think now the git installer does install automaticly add the env variables? i has updated my git to latet then the variable has changed automaticly. after i checked the env has changed i doesnt did anything in env for git

I also installed git long time ago so not sure about this but I think its important to mention that the path should be added to environment variables just in case someone faces error regarding it.

@defnax
Copy link
Author

defnax commented Jan 27, 2024

i added this infos for git path in setup, i was only surprised that my env was updated.

@defnax
Copy link
Author

defnax commented Feb 26, 2024

@csoler for merge

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