Skip to content

Comments

Assignment 1#1

Merged
dianayangs merged 8 commits intomainfrom
assignment
Oct 7, 2025
Merged

Assignment 1#1
dianayangs merged 8 commits intomainfrom
assignment

Conversation

@dianayangs
Copy link
Owner

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

made changes to assignment.sh to complete my assignment

What did you learn from the changes you have made?

I learned how to clone, branch, write commands and execute a .sh

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

There are many ways to code the solutions for the answers, I believe I used the simplest and effective code

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

when I executed my script, sometimes it would not work so I needed to modify, and I believe my final product will produce the correct results

How were these changes tested?

I executed my shell script again

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

question 4 originally I had fewer lines of code that worked when I tested in terminal but when I ran the full script it did not work so I modified it

Checklist

  • I can confirm that my changes are working as intended

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Autograder results

question status comment
Part 1 - Q1
Part 1 - Q2
Part 1 - Q3
Part 1 - Q4
Part 1 - Q5
Part 1 - Q6
Part 1 - Q7
Part 1 - Q8
Part 2 coworker-changes branch not found in commit history

Please address the issues listed above.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Autograder results

question status
Part 1 - Q1
Part 1 - Q2
Part 1 - Q3
Part 1 - Q4
Part 1 - Q5
Part 1 - Q6
Part 1 - Q7
Part 1 - Q8
Part 2

@dianayangs dianayangs merged commit 9eab881 into main Oct 7, 2025
2 checks passed
This was referenced Oct 7, 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