Skip to content

UofT-DSI | Shell - Assignment 1 - #1

Open
YariShukla wants to merge 5 commits into
mainfrom
assignment
Open

UofT-DSI | Shell - Assignment 1#1
YariShukla wants to merge 5 commits into
mainfrom
assignment

Conversation

@YariShukla

Copy link
Copy Markdown
Owner

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

I updated the assignment.sh script to automate the initial organization of client data files. The script now creates the required data directories, moves raw data into the correct location, organizes server, user, and event log files, removes files containing IP addresses for privacy, and creates an inventory file of processed data.

What did you learn from the changes you have made?

I learned how to use Bash commands for file and directory management, including creating folders, moving and copying files, deleting files, and generating file lists. I also practiced using Git branches, commits, and pull requests to manage and submit code changes.

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

I considered manually organizing the files, but using a Bash script provides a more efficient, repeatable, and automated solution that can be reused by the team.

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

I faced challenges understanding the correct file paths and Git workflow. I overcame these challenges by testing the script locally, verifying the directory structure, and using Git commands to track and submit my changes.

How were these changes tested?

I tested the script locally by running bash assignment.sh and verifying that the required directories were created, files were organized correctly, unwanted IP address files were removed, and the inventory file was generated.

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

Not applicable.

Checklist

  • I can confirm that my changes are working as intended

@Dmytro-Bonislavskyi Dmytro-Bonislavskyi 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.

Approved

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.

3 participants