Skip to content

Assignment#295

Closed
PSulf wants to merge 4 commits intoUofT-DSI:mainfrom
PSulf:assignment
Closed

Assignment#295
PSulf wants to merge 4 commits intoUofT-DSI:mainfrom
PSulf:assignment

Conversation

@PSulf
Copy link
Copy Markdown

@PSulf PSulf commented Apr 12, 2025

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

Completed assignment.sh to create project structure, unzip rawdata, sort log files into server/user/event folders, remove IP logs, and generate inventory file. Also merged coworker updates manually.

What did you learn from the changes you have made?

How to structure a shell script for setup automation, use wildcards and find, resolve Git conflicts, and test everything in a clean folder.

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

Thought of skipping the clean test directory but followed all steps as required.

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

Merge conflict in assignment.sh after pulling coworker branch. Fixed manually using VS Code, committed, and pushed. Also messed up PR target first time — re-did PR into correct fork branch.

How were these changes tested?

Created clean folder assignment_test_clean, copied script, ran it with bash assignment.sh, checked output folders and log files.

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

n/a

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.

2 participants