Skip to content

Update README.md#219

Closed
louischatriot wants to merge 1 commit into
expressjs:masterfrom
louischatriot:master
Closed

Update README.md#219
louischatriot wants to merge 1 commit into
expressjs:masterfrom
louischatriot:master

Conversation

@louischatriot

Copy link
Copy Markdown
Contributor

Added express-nedb-session to the list of session stores. It uses (as it name states) NeDB for data persistence.

@dougwilson

Copy link
Copy Markdown
Contributor

I know it's your module and all, but I'm slightly hesitant to add it, as the very tops says:

this library is not maintained anymore, please don't submit anymore pull requests except for bugfixes

Should people still be using it? Is there some kind of replacement people should go for instead?

@louischatriot

Copy link
Copy Markdown
Contributor Author

Right, the phrasing is not very clear. I mean that I'll not add any feature but still take bug reports. That's a warning I copy pasted on lots of projects without rephrasing it :) I could say it's feature-freezed instead if you prefer (not like there is any feature to add anyway). In any case the underlying db is an active project.

@dougwilson

Copy link
Copy Markdown
Contributor

Thanks, makes sense! You don't need to change the wording on your module, I just wanted to verify :) I have merged the PR.

dougwilson pushed a commit that referenced this pull request Oct 29, 2015
@louischatriot

Copy link
Copy Markdown
Contributor Author

Got it, thanks !

@expressjs expressjs locked and limited conversation to collaborators Dec 7, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants