professor-greebie / SENG8080-1-field_project

Other
21 stars 32 forks source link

#Add Arman #21

Closed ArmanShareef closed 1 year ago

ArmanShareef commented 1 year ago

@jjaramillimejia0773

ArmanShareef commented 1 year ago

@jjaramillomejia0773

ArmanShareef commented 1 year ago

@JJaramillomejia0773

JJaramillomejia0773 commented 1 year ago

LGTM @ArmanShareef

professor-greebie commented 1 year ago

@JJaramillomejia0773 Once you accept the request to join the repository as a contributor, you should be able to merge the pull request and close it.

JJaramillomejia0773 commented 1 year ago

@professor-greebie but this is close Please look up the picture attached

Screenshot 2023-05-24 121116

professor-greebie commented 1 year ago

It's fine that the issue is closed (you will need to close it anyway once it's merged). But this pull request needs merging, if you are ready. You should see a "merge pull request" button to complete the merge and add the changes (if you approve them).

JJaramillomejia0773 commented 1 year ago

@professor-greebie Could you show me which button it is I can't find it