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

Add key-fn library wide #67

Closed
wants to merge 13 commits into from
Closed

Conversation

CHNB128
Copy link

@CHNB128 CHNB128 commented Oct 8, 2020

This PR attempt to solve these problems: #62 #51

@weavejester
Copy link
Member

You seem to have a bunch of commits in that aren't relevant to the PR. Can you clean the branch up first?

@atomist
Copy link

atomist bot commented Oct 13, 2020

Commit messages of this repository should follow the seven rules of a great Git commit message, as mentioned in the project's contributing guidelines. It looks like there's a few issues with the commit messages in this pull request:

  • The commit message subject is over 50 characters.

@CHNB128
Copy link
Author

CHNB128 commented Oct 13, 2020

I will recreate this pr with clean commit history.
Sorry for that.

@CHNB128 CHNB128 closed this Oct 13, 2020
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