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

- (Bug Fix) Fixed typo in the definition of getWeeks, preventing its usage #150

Merged

Conversation

charlesbaynham
Copy link

As the description says, minor typo in the definition of getWeeks. Doesn't affect you unless you're making your own faces since this method is not used by default.

@objectivehtml
Copy link
Owner

I will get this merged as soon as I can. But it's helpful and makes things easier/quicker when people recompile the source and bump the version in the repo along with their pull request.

@charlesbaynham
Copy link
Author

No problem: I didn't because some people like doing versioning themselves.
I'll put those in when I'm at my computer.

On 21 Oct 2014, at 16:46, Justin Kimbrell notifications@github.com wrote:

I will get this merged as soon as I can. But it's helpful and makes things
easier/quicker when people recompile the source and bump the version in the
repo.


Reply to this email directly or view it on GitHub
#150 (comment).

@objectivehtml
Copy link
Owner

I really need to get document together for best practices when submitting pull request. I usually do the versioning, but sometimes I tend to forget and submit the pull request. And now that FlipClock is on npm, I have to publish it there and the versioning is strict and semantic. There are like 3 places the version has to change, in the npm config file, in bower, and within the FlipClock.Factory object. And then... I have to make a commit to the FlipClock's home page which is in Beanstalk to get the update version on the site (kind of a pain).

If anyone knows of a better way to handle the versioning and publishing new releases, I am open to suggestions.

@charlesbaynham
Copy link
Author

There, I've bumped the version and rebased.

As for versioning, I've not used it, but check out https://github.com/kswedberg/grunt-version. It looks like it just sets/lifts the version to/from package.json and then uses a regex search to push this to a given list of other files where it needs to go.

objectivehtml added a commit that referenced this pull request Nov 18, 2014
- (Bug Fix) Fixed typo in the definition of getWeeks, preventing its usage
@objectivehtml objectivehtml merged commit 880fe38 into objectivehtml:master Nov 18, 2014
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