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 info about a timer bug fix. #385

Merged
merged 3 commits into from Dec 20, 2012
Merged

Add info about a timer bug fix. #385

merged 3 commits into from Dec 20, 2012

Conversation

Kami
Copy link
Member

@Kami Kami commented Dec 20, 2012

Adds a missing changelog entry :)

@creationix
Copy link
Member

Maybe too verbose? Could this be shortened while still being descriptive?

@Kami
Copy link
Member Author

Kami commented Dec 20, 2012

@creationix Sure. Is 1a286b6 better?

I also referenced the pull request number in the message. I plan to open another PR which adds ticket references to other messages unless you don't like this idea.

@philips
Copy link
Contributor

philips commented Dec 20, 2012

@Kami Can you make it one line. People can reference the ticket if they want more details.

@Kami
Copy link
Member Author

Kami commented Dec 20, 2012

@philips Sure - 2258cdd. And yeah, one line should be sufficient if there is a ticket reference next to it (as long as the PR / ticket contains a useful description) :)

@philips
Copy link
Contributor

philips commented Dec 20, 2012

@Kami Thanks for adding this. Sorry I missed it, I obviously had a lot of stuff to sort through. 6 months of work!

philips added a commit that referenced this pull request Dec 20, 2012
Add info about a timer bug fix.
@philips philips merged commit f8a4e06 into luvit:master Dec 20, 2012
@Kami
Copy link
Member Author

Kami commented Dec 21, 2012

@philips No problem and thanks!

zhaozg pushed a commit to zhaozg/luvit that referenced this pull request Apr 2, 2019
Add info about a timer bug fix.
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

3 participants