Skip to content
This repository has been archived by the owner on Nov 15, 2019. It is now read-only.

feat: callback on missing localization #69

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

Conversation

egitto
Copy link

@egitto egitto commented Jan 5, 2018

I want to add a callback, so that instead of just sending missing localization errors I can, for instance, add the terms to a file.

@jsf-clabot
Copy link

jsf-clabot commented Jan 5, 2018

CLA assistant check
All committers have signed the CLA.

@egitto egitto changed the title Missinglocalization callback feat: callback on missing localization Jan 5, 2018
@egitto egitto force-pushed the missinglocalization-callback branch from 18dc565 to 2a6c85b Compare January 5, 2018 18:44
@codecov
Copy link

codecov bot commented Jan 5, 2018

Codecov Report

Merging #69 into master will increase coverage by 0.11%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #69      +/-   ##
==========================================
+ Coverage   95.06%   95.18%   +0.11%     
==========================================
  Files           4        4              
  Lines          81       83       +2     
  Branches       23       24       +1     
==========================================
+ Hits           77       79       +2     
  Misses          4        4
Impacted Files Coverage Δ
src/index.js 94.44% <100%> (+0.21%) ⬆️

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 930100a...23ed2a4. Read the comment docs.

@egitto egitto changed the base branch from master to next January 5, 2018 19:46
@egitto egitto changed the base branch from next to master January 5, 2018 19:54
@egitto egitto force-pushed the missinglocalization-callback branch from 2a6c85b to 23ed2a4 Compare March 22, 2018 18:32
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

2 participants