Skip to content
This repository has been archived by the owner on Oct 27, 2020. It is now read-only.

Update README.md #90

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

akhilbhardwaj23
Copy link

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update
  • documentation update

Motivation / Use-Case

When not using the default cache store, the read/write functions should use BJSON, similar, to how it has been done with the default read/write functions. Not using BJSON(buffer-json) throws errors with raw loaders.

Breaking Changes

Additional Info

When not using the default cache store, the read/write functions should use BJSON. Similar, to how it has been done with the default read/write functions. Not using BJSON(buffer-json) throws errors with raw loaders.
@jsf-clabot
Copy link

jsf-clabot commented Jul 31, 2019

CLA assistant check
All committers have signed the CLA.

README.md Outdated Show resolved Hide resolved
@mistic
Copy link
Collaborator

mistic commented Aug 1, 2019

Also please sign the contributor agreement @akhilbhardwaj23

Co-Authored-By: Tiago Costa <tiagoffcc@hotmail.com>
@Mordred
Copy link
Contributor

Mordred commented May 12, 2020

@mistic Can you merge? It is important piece of missing information in documentation.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants