Skip to content

shell assignment complete - #1

Open
CarlaKulcsar wants to merge 5 commits into
mainfrom
assignment
Open

shell assignment complete#1
CarlaKulcsar wants to merge 5 commits into
mainfrom
assignment

Conversation

@CarlaKulcsar

Copy link
Copy Markdown
Owner

UofT-DSI | Shell - Assignment 2

What changes are you trying to make?

Changed the assignment.sh shell script and modified it to create directories, move files and create a text file.

What did you learn from the changes you have made?

I have learned how to use git hub, shell scripts and my local terminal

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?

Branching correctly and making the correct pull request. The assignment itself was not hard.

How were these changes tested?

Ran line by line in my terminal and finally the whole script.

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

Checklist

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

@Dmytro-Bonislavskyi Dmytro-Bonislavskyi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hi, can you, please activate "Actions" and trigger the autograder?

Image

@CarlaKulcsar

Copy link
Copy Markdown
Owner Author

Hi Dmytro, thanks for your note. I am trying to trigger the autograder on my assignment branch, but it says that it is being 'skipped'. I am not sure how to fix this.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

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

@Dmytro-Bonislavskyi Dmytro-Bonislavskyi left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Well done! Approved. Now you can merge PR.

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.

3 participants