Skip to content

Comments

Merge pull request #1 from deeqa-s/assignment#2

Merged
deeqa-s merged 1 commit intoassignmentfrom
main
Nov 7, 2025
Merged

Merge pull request #1 from deeqa-s/assignment#2
deeqa-s merged 1 commit intoassignmentfrom
main

Conversation

@deeqa-s
Copy link
Owner

@deeqa-s deeqa-s commented Nov 7, 2025

Assignment

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

Merged with coworker changes

What did you learn from the changes you have made?

How to compare changes

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?

Just needed to review bash command arguments (eg adding -p to mkdir)

How were these changes tested?

tested the scripts to make sure expected directories and files were created/copied

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

Checklist

  • I can confirm that my changes are working as intended

@deeqa-s deeqa-s merged commit b357777 into assignment Nov 7, 2025
2 checks passed
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