search

Level / abstract-leveldown

An abstract prototype matching the leveldown API.
MIT License
146 stars 53 forks source link

Update changelog with TypeScript changes #199

Closed vweevers closed 6 years ago

vweevers commented 6 years ago

Would you mind fixing all capital letters for bulleted lists for the rest of the file?

Sure.

vweevers commented 6 years ago

@ralphtheninja on a related note, should we start all lines with a verb (e.g. "Add x", "Remove y")?

ralphtheninja commented 6 years ago

@ralphtheninja on a related note, should we start all lines with a verb (e.g. "Add x", "Remove y")?

Yeah, lets do that. I was thinking about it before. At least whenever it makes sense.

vweevers commented 6 years ago

@ralphtheninja for adding a release title and date to the changelog, is it OK to push to master? I'm not sure what the workflow for publishing should be.

ralphtheninja commented 6 years ago

@vweevers Yes I think so. When publishing I normally do npm version major|minor|patch and push to master and push up the tags. Now fixing the changelog is basically part of that workflow, so entirely ok since it's a trivial change.

vweevers commented 6 years ago

4.0.1