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

Y033 bind members up top #45

Merged
merged 1 commit into from
Feb 6, 2017

Conversation

phamdt
Copy link
Contributor

@phamdt phamdt commented Jan 30, 2017

my end goal is to address the functionality in #44. to become familiar with the code, i'd like to submit a series of PRs to john papa-fy the codebase.

@phamdt
Copy link
Contributor Author

phamdt commented Feb 5, 2017

@ealves are there any contribution guidelines i should follow or can we merge?

@phamdt
Copy link
Contributor Author

phamdt commented Feb 5, 2017

also since there will be multiple related PRs, we could merge them against an integration branch first before including it as part of one bigger release.

@ealves ealves merged commit d8f9590 into ealves:master Feb 6, 2017
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