Skip to content

Final: assignment#284

Closed
PSulf wants to merge 1 commit intoUofT-DSI:mainfrom
PSulf:assignment
Closed

Final: assignment#284
PSulf wants to merge 1 commit intoUofT-DSI:mainfrom
PSulf:assignment

Conversation

@PSulf
Copy link
Copy Markdown

@PSulf PSulf commented Apr 12, 2025

What changes are you trying to make?

I completed all steps for the Shell assignment by writing the correct commands into the assignment.sh file. This includes creating folders, moving files, copying specific logs, deleting sensitive ones, and listing final outputs.

What did you learn from the changes you have made?

This task helped me become more confident using basic shell commands. I practiced working with wildcards, organizing files, and chaining commands efficiently.

Was there another approach you were thinking about making?

At first, I thought I could copy files one by one, but using patterns like *server*.log and *ipaddr* made things much simpler and faster.

Were there any challenges?

Yes! I started working in the wrong folder at first. Then I realized I had to move everything into the shell_assignment folder and start over step-by-step — but that helped me understand Git and shell better, so no regrets.

How were these changes tested?

I tested all commands in Git Bash, used ls, cat, and grep to verify file movement and deletions, and confirmed everything with inventory.txt at the end.

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

The only issue was that I originally worked outside the right folder. Later I created shell_assignment, set the branch, cloned correctly, and redid the work there.

Checklist

  • I can confirm that my changes are working as intended.

@github-actions
Copy link
Copy Markdown

This pull request was made to the wrong repository. Please open it in your own fork instead. Refer to the Assignment Submission Guide for detailed instructions.

@github-actions github-actions bot closed this Apr 12, 2025
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.

1 participant