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

Remove trailing whitespaces #51

Closed

Conversation

zhongyi-zhang
Copy link
Collaborator

Resubmit a PR to cover #2.

@cfdreddbot
Copy link

Hey zhongyi-zhang!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

@zhongyi-zhang
Copy link
Collaborator Author

@mattmcneeney could you take a look at the "Invalid .pullapprove.yml" issue? Seems pullapprove.com doesn't reflect the latest change in .pullapprove.yml.

@mattmcneeney
Copy link
Contributor

@zhongyi-zhang I've found the problem - @leonwanghui was not a collaborator of this repo. I've fixed this, so once @leonwanghui has accepted the invite, I'll get pullapprove to try again.

@leonwanghui
Copy link
Collaborator

Thanks @mattmcneeney
Hi @zhongyi-zhang I have accepted the invitation, please reopen this PR to see if it works.

@cfdreddbot
Copy link

Hey zhongyi-zhang!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

@zhongyi-zhang
Copy link
Collaborator Author

It seems reopening doesn't work. I'll try creating a new one.

@zhongyi-zhang
Copy link
Collaborator Author

The new PR #53 doesn't work, either...

@mattmcneeney
Copy link
Contributor

@zhongyi-zhang I had to restart pullapprove again. All working now I think! cc @leonwanghui

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.

None yet

4 participants