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

Incoming pull request #3

Open
tgeek77 opened this issue Mar 4, 2024 · 2 comments
Open

Incoming pull request #3

tgeek77 opened this issue Mar 4, 2024 · 2 comments

Comments

@tgeek77
Copy link

tgeek77 commented Mar 4, 2024

Hi, I just wanted to say that it is awesome that you're doing this! However, there are a lot of issues with your material. I am going though your csv file and making a lot of changes. You can see my current status here.

I spent the last 6 years as a Linux course developer and occasional instructor. Please let me know if you have any questions.

@cipher387
Copy link
Owner

It's just a text version of the picture. I'm not going to change anything about it.

@tgeek77
Copy link
Author

tgeek77 commented Mar 8, 2024

You've got some things that could use some work. For example, you have cat>ne­w_f­ile.txt instead touch ne­w_f­ile.txt. Sure, they technically do the same thing, but the touch command is the most direct way to do this while the cat command is kind of a hack. If you're not interested in input, that's fine. I won't bother you. I thought you might want a second pair of eyes to review your fIles.

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

No branches or pull requests

2 participants