Skip to content

Edit intro section. Adjust/refine wording, but not the core themes.#1

Merged
esteban-url merged 2 commits intoesteban-url:test-in-github-actionsfrom
keithtelliott:kte-test-in-github-actions
Nov 22, 2022
Merged

Edit intro section. Adjust/refine wording, but not the core themes.#1
esteban-url merged 2 commits intoesteban-url:test-in-github-actionsfrom
keithtelliott:kte-test-in-github-actions

Conversation

@keithtelliott
Copy link
Copy Markdown
Collaborator

@esteban-url I climbing the GitHub learning curve. I have submitted my own PRs, but never have jumped into help with another PR. So, I'm figuring out the workflow.

I tried to push to your fork, but I'm getting 403 errors. So my latest attempt: Create my own fork, checkout your branch, rename it, push it to my forked repo, then submit a PR to you.

Is this how it should be done?

One day I'll know these GitHub flows :)

@keithtelliott
Copy link
Copy Markdown
Collaborator Author

@esteban-url I performed minor edits to the body of the article. Next up...

Go-Do: I will actually step through the guide (I started, but need more time).

Go-Do: Add concluding remarks at the end of the doc (me or you).

@esteban-url
Copy link
Copy Markdown
Owner

Thank you so much for your help @keithtelliott! the edits are great! please let me know once you step through the guide, maybe it needs some refinement or further explanation.

@esteban-url
Copy link
Copy Markdown
Owner

also, I've added you as a contributor on my repo, you should be able to create a PR directly on it now.

@esteban-url esteban-url merged commit 0f5ea3e into esteban-url:test-in-github-actions Nov 22, 2022
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