Skip to content

Add bash commands to assignment_script.sh to process 'DSI Consulting'… - #1

Open
Keith-88h wants to merge 5 commits into
mainfrom
assignment_shell
Open

Keith-88h wants to merge 5 commits into
mainfrom
assignment_shell

Conversation

@Keith-88h

Copy link
Copy Markdown
Owner

… data

TITLE: UofT-DSI | Unix Shell, Git & GitHub - Assignment 1

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

Updated 'assignment_script.sh' to completed the 'Unix Shell, Git & GitHub - Assignment 1' module.

What did you learn from the changes you have made?

Refreshed my memory on various bash commands, such as 'mkdir', 'mv', 'cp', and 'rm', and being aware of the working directory when programming.

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

  1. When creating sub-directories such as 'server_logs', I considered first 'cd'ing into the 'data' folder, so the path could be shorter (and not repeated); then 'cd'ing back to the parent folder, once the commands were complete. However, since this technique was not explicitly stated in the instruction, I did not do it.
  2. For item 8, I considered 'ls' without the '-1', but the output in data/inventory.txt looks is more concise when it is outputted as one file per line. (also easier to parse in programs such as 'wc'.

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

When I first ran the script, the 'data/processed/event_logs/ folder was empty. I found a typo in the script, which specified, incorrecly to cp data/raw/*event*logs. This was corrected to cp data/raw'*event*log.

How were these changes tested?

I tested the commands in a separate terminal, as I was updating the script. Then, after deleting the newly created directories and files, I ran the script and confirmed the output.

A reference to a related issue in your repository (if applicable)

n/a

Checklist

  • I can confirm that my changes are working as intended
  • I tested the 'assignment_script.sh' on my computer and it worked as expected

@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown

Autograder results

question status comment
Part 1 - Q1 ✅
Part 1 - Q2 ✅
Part 1 - Q3 ✅
Part 1 - Q4 ✅
Part 1 - Q5 ✅
Part 1 - Q6 ✅
Part 1 - Q7 ✅
Part 1 - Q8 ✅
Part 2 ❌ coworker-changes branch not found in commit history

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Autograder checks failed. Please address the issues listed in the results comment.

@github-actions

github-actions Bot commented Nov 8, 2026

Copy link
Copy Markdown

Autograder results

question status
Part 1 - Q1 ✅
Part 1 - Q2 ✅
Part 1 - Q3 ✅
Part 1 - Q4 ✅
Part 1 - Q5 ✅
Part 1 - Q6 ✅
Part 1 - Q7 ✅
Part 1 - Q8 ✅
Part 2 ✅

All autograder tests passed. This submission is approved by the autograder.

@Keith-88h Keith-88h left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Coworker-changes merged

  • all conflicts resolved
    • updated command to populate README.md file,
    • removed invalid command rf -rf and non-functional commands for appending text.

@tianyi21 tianyi21 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi Keith,

Thank you for submitting your Unix/Shell/Git assignment.

✅ Your assignment is complete. Thank you for your participation.

Best,
Tianyi - Unix/Shell/Git Learning Support

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.

4 participants