Skip to content
This repository has been archived by the owner on Jan 18, 2024. It is now read-only.

Remove exp.json support #2017

Merged
merged 2 commits into from
Apr 28, 2020
Merged

Conversation

EvanBacon
Copy link
Contributor

No description provided.

@codecov-io
Copy link

codecov-io commented Apr 28, 2020

Codecov Report

Merging #2017 into master will not change coverage.
The diff coverage is n/a.

Flag Coverage Δ
#babelPresetCli 100.00% <ø> (ø)
#config 60.90% <ø> (ø)
#expoCli 32.09% <ø> (ø)
#expoCodemod 83.81% <ø> (ø)
#jsonFile 65.49% <ø> (ø)
#metroConfig 58.83% <ø> (ø)
#packageManager 16.67% <ø> (ø)
#plist 70.64% <ø> (ø)
#pwa 34.32% <ø> (ø)
#schemer 69.88% <ø> (ø)
#uriScheme 32.05% <ø> (ø)
#webpackConfig 53.41% <ø> (ø)
#xdl 23.37% <ø> (ø)
@@           Coverage Diff           @@
##           master    #2017   +/-   ##
=======================================
  Coverage   44.66%   44.66%           
=======================================
  Files         112      112           
  Lines        4777     4777           
  Branches     1148     1148           
=======================================
  Hits         2133     2133           
  Misses       1858     1858           
  Partials      786      786           

Copy link
Contributor

@fson fson left a comment

Choose a reason for hiding this comment

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

Nice! 👍

Would be good to update/remove these comments too:

// Modify exp.json

// Update exp.json/app.json

Copy link
Member

@brentvatne brentvatne left a comment

Choose a reason for hiding this comment

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

rip

@EvanBacon EvanBacon merged commit 0d49041 into master Apr 28, 2020
@EvanBacon EvanBacon deleted the @evanbacon/xdl/remove-exp.json-support branch April 28, 2020 17:07
@EvanBacon EvanBacon self-assigned this Apr 28, 2020
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.

4 participants