Skip to content

Assignment done - #1

Open
HaroonV12 wants to merge 5 commits into
mainfrom
assignment
Open

Assignment done#1
HaroonV12 wants to merge 5 commits into
mainfrom
assignment

Conversation

@HaroonV12

Copy link
Copy Markdown
Owner

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

I created a shell script that organizes a project directory by creating the required folders, moving the raw data into the correct location, copying log files into categorized subdirectories, removing files containing IP addresses for privacy, generating an inventory of the processed files, and then at the end displaying a completion message.

What did you learn from the changes you have made?

It gave me practice on how to use common commands we learnt such as mkdir, mv, cp, rm, touch. I also learned how to run a script through vsc and automate file and directory management using a shell script.

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

I considered another method I found online by creating the inventory.txt file directly using output redirection, but I used touch first because that was the approach taught in class.

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

One challenge was understanding how to create and populate inventory.txt. I initially wasn't sure whether I needed to create the file first, but I learned that using touch followed by the find command correctly creates and fills the file.

How were these changes tested?

I executed the shell script and verified that all directories and files were created correctly, the log files were copied into the appropriate folders, the IP address files were removed, and the script completed without any errors.

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

Checklist

  • I can confirm that my changes are working as intended

@KasraVakiloroayaei KasraVakiloroayaei 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.

A1: Incomplete

Good work thus far,

Please address the comments from the auto grader to have all of them completed.

@KasraVakiloroayaei

Copy link
Copy Markdown

Hello,

Below you can see attached screen shot that shares the parts the auto grader is suggesting to fix.

image

@KasraVakiloroayaei KasraVakiloroayaei 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.

A1: Complete

great work!

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