Skip to content

Brigid Goulem Assignment 1 #595

Closed
BGoulem wants to merge 6 commits intoUofT-DSI:mainfrom
BGoulem:assignment
Closed

Brigid Goulem Assignment 1 #595
BGoulem wants to merge 6 commits intoUofT-DSI:mainfrom
BGoulem:assignment

Conversation

@BGoulem
Copy link

@BGoulem BGoulem commented Mar 4, 2026

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

I used a bash script to process raw data. This looked like creating a script for new files, copying files into the appropriate directories, removing files with the IP addresses, and creating a txt file with a list of the files.

What did you learn from the changes you have made?

The biggest thing I learned was the importance of making the changes in the script editor (Visual Studio Code) and not directly in the terminal. This was the biggest barrier. I also learned more practical things like using wildcard and how to properly use file paths.

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

I don't think so, it was pretty straightforward in the end.

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

I made the mistake of entering the code into the terminal rather than script editor, which obviously meant that the script wasn't saving. I spent a lot of time trying to troubleshoot this.

How were these changes tested?

I tested the changes by continually running the bash assignment.sh to ensure that the code was running without errors, and then cross-referencing with the assignment file to check that the intended changes were happening. If they weren't I returned to the script and edited it.

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

Checklist

  • I can confirm that my changes are working as intended

Brigid Goulem added 6 commits March 3, 2026 12:27
…e data directory has been made, but no log files have yet been added.

:wq

 Please enter the commit message for your changes. Lines starting
…ta. I have not yet moved the raw data into the files.
… is the script up to step 8.

Please enter the commit message for your changes. Lines starting
…e files have been organized and there is a txt file with a list of all log files.
@github-actions
Copy link

github-actions bot commented Mar 4, 2026

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 Mar 4, 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