Skip to content

Finished assignment#299

Closed
code-me-matcha wants to merge 2 commits intoUofT-DSI:mainfrom
code-me-matcha:coworker-changes
Closed

Finished assignment#299
code-me-matcha wants to merge 2 commits intoUofT-DSI:mainfrom
code-me-matcha:coworker-changes

Conversation

@code-me-matcha
Copy link
Copy Markdown

@code-me-matcha code-me-matcha commented Apr 12, 2025

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

I added code to complete the assignment

What did you learn from the changes you have made?

I improved my ability to navigate using command line, and understand the structure of the files

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

I took several approaches - for me, it was easier to run every command after I typed it so I created a directory in the shell and ran each command step by step before finalizing it in the VS code.

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

The first few times, I kept unzipping the file multiple times until I understood the direction to open a clean directory to run the assignment.sh in each time.

How were these changes tested?

I made a clean directory each time and tested it.

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

Thanks for your contribution! 🎉

Please remember to tag or request a review from the DSI team. Give us up to 72 hours to review your pull request. We appreciate your patience and efforts.

@dtxe
Copy link
Copy Markdown
Member

dtxe commented Apr 14, 2025

This pull request was made to the wrong repository. Please open it in your own fork instead. Refer to the Assignment Submission Guide for detailed instructions.

@dtxe dtxe closed this Apr 14, 2025
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.

2 participants