Skip to content

This is my assignment#640

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

This is my assignment#640
cdanss wants to merge 1 commit intoUofT-DSI:mainfrom
cdanss:assignment

Conversation

@cdanss
Copy link
Copy Markdown

@cdanss cdanss commented Apr 15, 2026

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

Created directories, moved data, organized log files into processed subdirectories, removed IP address files for user privacy, and generated a file inventory.

What did you learn from the changes you have made?

mkdir requires parent directories to exist before creating subdirectories

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

Could have used mkdir -p data/processed/{server_logs,user_logs,event_logs} as a one-liner instead of three separate mkdir calls

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

data/processed subdirectories failed to create because the parent directory didn't exist yet. I had a lot of trouble trying to merge and resolve the conflicts

How were these changes tested?

By running bash assignment.sh

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

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. If you are a participant, please close it and open it in your own fork instead. Refer to the Assignment Submission Guide for detailed instructions.

@github-actions github-actions Bot closed this Apr 15, 2026
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