Skip to content

docs(readme): add credentials setup instructions#6

Merged
febinbellamy merged 2 commits intofebinbellamy:mainfrom
GabrielgsdCIUwU:patch-1
Mar 5, 2025
Merged

docs(readme): add credentials setup instructions#6
febinbellamy merged 2 commits intofebinbellamy:mainfrom
GabrielgsdCIUwU:patch-1

Conversation

@GabrielgsdCIUwU
Copy link
Copy Markdown
Contributor

Added instructions for creating a credentials.js file in the project root, including clientId and clientSecret setup. Also included a link to GitHub's official documentation on creating GitHub apps.

Added instructions for creating a `credentials.js` file in the project root, including `clientId` and `clientSecret` setup. Also included a link to GitHub's official documentation on creating GitHub apps.
@febinbellamy
Copy link
Copy Markdown
Owner

Hi Gabriel, thanks so much for the PR - your changes look great. I’ll need to make a small tweak so that it integrates a little bit better with step 4, and then we'll be all set to merge!

@GabrielgsdCIUwU
Copy link
Copy Markdown
Contributor Author

Hi Febin, that sounds good to me ^^

Co-authored-by: Febin Bellamy <febinbellamy@gmail.com>
@febinbellamy
Copy link
Copy Markdown
Owner

febinbellamy commented Mar 5, 2025

Hi Gabriel, please fetch the latest changes by running the following commands:

git pull origin patch-1
git push origin patch-1

This will update your local patch-1 branch with the latest changes. Once you've pulled these updates, we will be able to see them in the pull request. After that, I'll go ahead and merge the PR. Let me know if you have any questions or if you encounter any issues updating your branch. Thanks!

@GabrielgsdCIUwU
Copy link
Copy Markdown
Contributor Author

GabrielgsdCIUwU commented Mar 5, 2025

Hi Febin, I just updated the latest changes directly from Github. Let me know if there's anything else you need me to do.
Thanks!

@febinbellamy febinbellamy merged commit 7589a1e into febinbellamy:main Mar 5, 2025
@GabrielgsdCIUwU GabrielgsdCIUwU deleted the patch-1 branch March 5, 2025 22:43
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