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.

Sign up for GitHub

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

Jump to bottom

unsynchronize recentf saves #112

Merged
merged 1 commit into from
Jul 15, 2018
Merged

unsynchronize recentf saves #112

merged 1 commit into from
Jul 15, 2018

Conversation

cg505
Copy link
Contributor

@cg505 cg505 commented Jul 7, 2018

Hopefully avoids all future recentf lock-getting issues. Closes #48.

@cg505 cg505 requested a review from rye July 7, 2018 19:02
@cg505
Copy link
Contributor Author

cg505 commented Jul 15, 2018

@rye if you want to give this a quick review and merge, we'll be ready to release. no rush

rye
rye approved these changes Jul 15, 2018
View reviewed changes
Copy link
Member

@rye rye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've also seen quoted strings instead of a number fo seconds, but this is fine.

@cg505 cg505 merged commit 299463b into master Jul 15, 2018
@rye
Copy link
Member

rye commented Jul 15, 2018

@cg505 I will release when I get down to my coffee shop of choice

@rye rye mentioned this pull request Jul 15, 2018
Merged
@rye rye added this to the Version 0.2.0 milestone Jul 15, 2018
@rye rye deleted the recentf-fix branch July 16, 2018 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rye rye rye approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Version 0.2.0
Development

Successfully merging this pull request may close these issues.
2 participants
@cg505 @rye