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

Adding an increase & decrease rate for the button, fixing demo and project settings #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kmiloaguilar
Copy link

No description provided.

…Adding increase and decrease property. Updating project settings to avoid conflicts.
@devxoul
Copy link
Contributor

devxoul commented Jul 28, 2016

@kmiloaguilar, Thanks for your work. Could you please separate the core implementation? There're lot of unreleated diffs.

@kmiloaguilar
Copy link
Author

@devxoul Can you me more specific? How do you prefer to separate with core implementation? For now the new properties if they are not set it will still do the same. The Pod folder I removed all since not required over here. And the Demo is the only one. I could create a separate buttons for the demo if that is the separation you want.

@kmiloaguilar
Copy link
Author

Also the Demo was failing when doing a clean install, mostly because of the Pods. Adding instructions to run pod install can help too.

@devxoul
Copy link
Contributor

devxoul commented Jul 28, 2016

I mean, 1 PR (or 1 commit) should have 1 implementation. Your PR has only a commit that contains 'Adding an increase & decrease rate for the button' and 'fixing demo and project settings'.

@kmiloaguilar
Copy link
Author

ok so you want 1 PR for adding new properties to the button and another PR to fix the demo and project settings ?

@devxoul
Copy link
Contributor

devxoul commented Jul 28, 2016

@kmiloaguilar Exactly! Thanks for your understanding to keep commit & PR log clean 😄

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