Replacing setx in the readme.md
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 3k
- Forks
- 300
- Avg merge
- 19h 12m
- Merged PRs (30d)
- 1
Description
Hello, I noticed that the readme file recommends using setx but setx is old and it can only handle 1024 characters in the path variable. If you have more than 1024 characters in your path variable and try to use setx, your environment variable will get corrupt.
Since it mentions shell I'm assuming we're already using power shell or windows terminal from the beginning so I recommend something like this command instead of setx
[Environment]::SetEnvironmentVariable("Path", $env:Path +";C/folder directory", "Machine")
I know setx is easier to use but it's bit me more times than I can count
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in readme.md and find the instructions that recommend using setx. Replace that recommendation with the safer PowerShell environment-variable command proposed in the issue, then review the rendered README to confirm the guidance is clear and no longer recommends setx.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- powershell
- Domain
- documentation
- Issue type
- Documentation
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Quiet
- Clarity
- Clearly specified
- Newbie friendliness
- 78/100