Skip to content

Conversation

patrickkang
Copy link
Contributor

👋 This PR fixes a very small spelling mistake.

@netlify
Copy link

netlify bot commented May 4, 2022

Deploy Preview for pensive-meitner-faaeee ready!

Name Link
🔨 Latest commit 11625aa
🔍 Latest deploy log https://app.netlify.com/sites/pensive-meitner-faaeee/deploys/62730d24bd11b90008aaead7
😎 Deploy Preview https://deploy-preview-327--pensive-meitner-faaeee.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@eak12913 eak12913 mentioned this pull request May 4, 2022
@eak12913
Copy link
Contributor

eak12913 commented May 4, 2022

Hey @patrickkang, thanks so much for the fix. Could you please do me a favor and merge this branch into your current branch? We have some automated tasks which need to run against the changes you're making. Since these tasks can't access your forked repo, I manually pushed your changes to our own branch, had our tasks run and now, if you merge from this branch we should be able to merge.

I'm sorry for the extra step, I understand that it's unfortunate given the simple fix nature of the PR. I can also make the changes myself - but then you won't get credit for the change.

@patrickkang
Copy link
Contributor Author

Hi @eak12913. Sure, no problem. I just merged the branch into mine.

Copy link
Contributor

@ebeneliason ebeneliason left a comment

Choose a reason for hiding this comment

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

LGTM. Sorry for the delay on getting this one in.

@ebeneliason ebeneliason merged commit 6758258 into gruntwork-io:master May 9, 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.

3 participants