Skip to content

Comments

Merge pull request #1 from Empress-star/assignment#2

Closed
Empress-star wants to merge 1 commit intoassignmentfrom
main
Closed

Merge pull request #1 from Empress-star/assignment#2
Empress-star wants to merge 1 commit intoassignmentfrom
main

Conversation

@Empress-star
Copy link
Owner

Completed step 1 of the assignment

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

Added the code to the script to enact the changes that were requested

What did you learn from the changes you have made?

I learned to use bash to create files needed and to use wildcards in commands when copying files and removing certain ones

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

no

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

I was stuck on how to create the inventory file but I tested some different commands and ended up using the ls command with a wildcard so that it would print all the contents of the subfiles

How were these changes tested?

I did a clean test by creating another file and running the script there, ensuring that it worked as intentded

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

Checklist

  • [ x] I can confirm that my changes are working as intended

Completed step 1 of the assignment
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