Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update paths.md: fix typo #138

Merged
merged 1 commit into from
Sep 26, 2023
Merged

Update paths.md: fix typo #138

merged 1 commit into from
Sep 26, 2023

Conversation

corneliusroemer
Copy link
Contributor

No description provided.

@google-cla
Copy link

google-cla bot commented Sep 23, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@braydonk
Copy link
Collaborator

If you are willing to fill out the CLA I will merge this, if you're not interested in that I can make the fix myself. I appreciate the PR thanks for catching this!

@corneliusroemer
Copy link
Contributor Author

@braydonk I've already filled out the CLA - somehow the bot didn't reply again after I did but you can see the CI :)

image

@braydonk
Copy link
Collaborator

Ah sorry, I didn't notice that since it doesn't show up in the maintainer "approve and run CI" view 😅

Thank you!

@braydonk braydonk merged commit fd5c0af into google:main Sep 26, 2023
5 checks passed
@corneliusroemer
Copy link
Contributor Author

Ah sorry, I didn't notice that since it doesn't show up in the maintainer "approve and run CI" view 😅

Begging for an issue to be filed somewhere 😜

@braydonk
Copy link
Collaborator

braydonk commented Sep 26, 2023

Should have taken a screenshot while I had it! Not the first time I've made the same mistake with that UX though, I'll catch it next time

@corneliusroemer
Copy link
Contributor Author

To be fair to whoever wrote that cla bot, it does mention that at the bottom, though easily missed:

For the most up to date status, view the checks section at the bottom of the pull request.

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