Skip to content

Assignment script added#630

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

Assignment script added#630
milassc wants to merge 1 commit intoUofT-DSI:mainfrom
milassc:assignment

Conversation

@milassc
Copy link
Copy Markdown

@milassc milassc commented Apr 13, 2026

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

Initiating the pull request from the assignment branch where I have updated the assignment shell script.

What did you learn from the changes you have made?

Learned how to work with shell scripts particularly with the files moving and manipulations.

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

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

Though the requirement was simple, I have come across with some issues. Did not realize that the DSI script would be doing the unzipping part of the raw data. I had scripted for unzipping and later realized it's available part of DSI script. Though the individual scripts were running fine, the files are not moving and hence the final result was not coming as intended. After a thorough investigation, it was found that the DSI script was changing the directory to 'New Project', so the 'Data' folder was not getting created as designed. corrected the code and tested successfully.

How were these changes tested?

Ran the assigment.sh script from the terminal and the Inventory.txt is created successfully with User, Server and Event folder file names.

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 13, 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