use with_metadata argument as switch

[adbar]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.77%. Comparing base (04c2fdf) to head (261c919).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #613      +/-   ##
==========================================
- Coverage   97.94%   97.77%   -0.18%     
==========================================
  Files          21       21              
  Lines        3501     3499       -2     
==========================================
- Hits         3429     3421       -8     
- Misses         72       78       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[fortyfourforty]

It is a breaking change. I lost 30k of title extraction due to this change.
Not to complain, but why make with_metadata=False the default?

For normal users, more data is a good thing as metadata also provides useful information. For advances users, they know how to turn off metadata extraction for speed.

I suggest making the default True. just my 0.2 cents.