Skip to content

updated 100 Automations Overview link#1014

Closed
qiqicodes wants to merge 1 commit intogh-pagesfrom
1004-update-100-automations-project-overview-link
Closed

updated 100 Automations Overview link#1014
qiqicodes wants to merge 1 commit intogh-pagesfrom
1004-update-100-automations-project-overview-link

Conversation

@qiqicodes
Copy link
Copy Markdown
Member

Fixed #1004 Updated 100 Automations Project Overview link

@jbubar
Copy link
Copy Markdown
Member

jbubar commented Feb 7, 2021

Good work! you did everything right except for one thing. The code looks great, but since this is also about learning the git flow, I think it might be best to redo it.

It appears like you did not push from your own fork.

check out this part of the readme to set up git. Message me if you get stuck.
https://github.com/hackforla/website#step-2-fork-the-repository

@jbubar
Copy link
Copy Markdown
Member

jbubar commented Feb 7, 2021

This one is yours and it is missing the your fork.
Screen Shot 2021-02-07 at 3 50 59 PM
Look at how in daniel's pr you can see that hackforla: is before the gh-pages branch and his github username is before the branch he is merging
Screen Shot 2021-02-07 at 3 51 13 PM

@qiqicodes
Copy link
Copy Markdown
Member Author

Properly compared the code with new pull request #1015

@qiqicodes qiqicodes closed this Feb 8, 2021
@akibrhast akibrhast deleted the 1004-update-100-automations-project-overview-link branch April 10, 2021 23:21
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.

Update the 100 Automations project overview (one sheet) link

2 participants