Skip to content
This repository was archived by the owner on Jul 25, 2026. It is now read-only.

updated context implementation to use new React Context Api. - #256

Open
dipunm wants to merge 3 commits into
masterfrom
contextapi
Open

updated context implementation to use new React Context Api.#256
dipunm wants to merge 3 commits into
masterfrom
contextapi

Conversation

@dipunm

@dipunm dipunm commented Jul 11, 2018

Copy link
Copy Markdown
Contributor

Not sure why there was a package.json change. @matteofigus @nickbalestra maybe you could advise?

@codecov

codecov Bot commented Jul 11, 2018

Copy link
Copy Markdown

Codecov Report

Merging #256 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #256      +/-   ##
==========================================
- Coverage   96.04%   96.03%   -0.02%     
==========================================
  Files          19       19              
  Lines         253      252       -1     
  Branches       31       31              
==========================================
- Hits          243      242       -1     
  Misses         10       10
Impacted Files Coverage Δ
...late-react-compiler/lib/reactOCProviderTemplate.js 100% <ø> (ø) ⬆️
packages/oc-template-react/lib/render.js 95.23% <ø> (-0.22%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e9bb2e5...12286c6. Read the comment docs.

@matteofigus matteofigus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just a little one, can you also rebase with the snapshot fixes

Comment thread package.json Outdated
Comment thread lerna.json Outdated
@dipunm

dipunm commented Nov 10, 2018

Copy link
Copy Markdown
Contributor Author

@matteofigus done thanks

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.

2 participants