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

Add commit with vim info to the instructions #16

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

Conversation

taylorshephard
Copy link
Contributor

  • Added info about committing with vim to lab 8

Copy link

@ptbarnum4 ptbarnum4 left a comment

Choose a reason for hiding this comment

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

I think the mention of vim could happen in an earlier prompt (it can also be mentioned here)

Left additional suggestions on the code

docs/lab_08.html Outdated Show resolved Hide resolved
docs/lab_08.html Outdated
# modified: hello.js
#
</pre>
<p>On the first line, you would enter the commit message for example: &#8220;Using process.argv&#8221;. Save the file and exit the editor. You then would see:</p>

Choose a reason for hiding this comment

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

Maybe include information, or at least reference how to

  • Enter Insert mode
  • Save and exit
  • Exit without saving

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay I added a list of commands that explain how to do that

@taylorshephard
Copy link
Contributor Author

I put it on lab 8 because lab 7 and 8 are labeled to be about committing. I can also put it on lab 7 instead @ptbarnum4
Screenshot 2024-09-11 at 3 35 57 PM

@ptbarnum4
Copy link

I put it on lab 8 because lab 7 and 8 are labeled to be about committing. I can also put it on lab 7 instead @ptbarnum4 Screenshot 2024-09-11 at 3 35 57 PM

@taylorshephard I suggested putting it earlier because it references the previous step

docs/lab_08.html Outdated Show resolved Hide resolved
docs/lab_08.html Outdated Show resolved Hide resolved
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