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

Update CheckboxControl from apache/master #85

Merged
merged 3 commits into from
Aug 15, 2018

Conversation

kristw
Copy link

@kristw kristw commented Aug 14, 2018

The previously merged cherry-pick requires CheckboxControl from this giant PR, which is not merged yet, so I am copying this file over.
apache#5186

@graceguo-supercat @michellethomas @john-bodley @williaster

@kristw kristw changed the title [WIP] Update from apache/master: <description> Update CheckboxControl from apache/master Aug 14, 2018
@codecov-io
Copy link

codecov-io commented Aug 14, 2018

Codecov Report

Merging #85 into airbnb-master will increase coverage by <.01%.
The diff coverage is 83.33%.

Impacted file tree graph

@@               Coverage Diff                @@
##           airbnb-master     #85      +/-   ##
================================================
+ Coverage           63.2%   63.2%   +<.01%     
================================================
  Files                351     351              
  Lines              22040   22044       +4     
  Branches            2468    2469       +1     
================================================
+ Hits               13931   13934       +3     
- Misses              8097    8098       +1     
  Partials              12      12
Impacted Files Coverage Δ
...rc/explore/components/controls/CheckboxControl.jsx 89.47% <83.33%> (-3.86%) ⬇️

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 ca006c6...51a4bdd. Read the comment docs.

@kristw kristw merged commit b6ce0c1 into airbnb:airbnb-master Aug 15, 2018
@kristw kristw deleted the kristw/fix-checkbox branch August 22, 2018 18:14
michellethomas pushed a commit that referenced this pull request Dec 1, 2021
* feat: Allow any http METHOD

* Add method to RequestConfig

* linting

* fix tests
michellethomas pushed a commit that referenced this pull request Dec 1, 2021
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.

3 participants