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

Issue#29 #51

Closed
wants to merge 2 commits into from
Closed

Conversation

SaumyaBhushan
Copy link
Contributor

What does this PR do?

Add content(s)

Description

added new commands for nvm

Screenshots or Links

If applicable, add screenshots or a modified UI link to help understand your contribution.

Additional context

#29

Checklist:

Follow-up

  • Check the status of GitHub Actions and resolve any reported warnings!

@crescentpartha crescentpartha added hacktoberfest Hacktoberfest Participation hacktoberfest-2022 Hacktoberfest-2022 Participation 📝review-pull-request Reviewing your pull request labels Oct 20, 2022
Copy link
Owner

@crescentpartha crescentpartha left a comment

Choose a reason for hiding this comment

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

Request for changes.

Comment on lines +45 to +52
| Command | Description |
|---------|-----------------------------|
| `Ctrl + W` | Close the current window |
| `Shift + Tab`| Set Focus on Side Bar |
| `Ctrl + N`| Duplicate the File Explorer |
| `Crtl + Shift + N`| Create a New Folder |
| `Alt + ←`| Navigate to previous folder |
| `Ctrl + L` | Navigate to the address bar |
Copy link
Owner

Choose a reason for hiding this comment

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

These commands are already merged via your first PR. Please, remove your windows-cheatssheet.md file's modification and only commit for nvm-cheatsheet.md file. @SaumyaBhushan

Comment on lines +27 to +29
| `nvm -v` | Check if nvm is installed |
| `nvm uninstall <version>` | Un-Install a specific version |
| `nvm ls` | Show versions available locally |
Copy link
Owner

Choose a reason for hiding this comment

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

Only these modifications should be added to this PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok let me check

@crescentpartha crescentpartha added modifications-needed Modify the code waiting-for-changes Waiting for require changes labels Oct 20, 2022
@SaumyaBhushan
Copy link
Contributor Author

Hi @crescentpartha I have opened a new pull request can you review that.

@crescentpartha crescentpartha added duplicate This issue or pull request already exists invalid This doesn't seem right labels Oct 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists hacktoberfest Hacktoberfest Participation hacktoberfest-2022 Hacktoberfest-2022 Participation invalid This doesn't seem right modifications-needed Modify the code 📝review-pull-request Reviewing your pull request waiting-for-changes Waiting for require changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants