Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Write config to & read config from JSON file (+ Chromium) #45

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

Conversation

kchawla-pi
Copy link
Owner

  • Key for profiles collected is now a tuple of Browser name& profile name, not just profile name.
  • CHromium is now an accepted choice when manually adding browsers, but not auto-discovered.

…of-browsers into profile-paths-dict-change

* 'master' of https://github.com/kchawla-pi/united-states-of-browsers:
  Convert db_merge class to function: OOP has no benefit here  (#44)
  Tweak web UI - full page; sticky search box; no ext libs (#43)
 - Chromium has been added as supported but not auto-discovered, to enable testing this feature.
@codecov-io
Copy link

Codecov Report

Merging #45 into master will decrease coverage by 1.22%.
The diff coverage is 82.35%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #45      +/-   ##
==========================================
- Coverage    94.7%   93.48%   -1.23%     
==========================================
  Files          11       11              
  Lines         378      399      +21     
==========================================
+ Hits          358      373      +15     
- Misses         20       26       +6
Impacted Files Coverage Δ
united_states_of_browsers/db_merge/browser.py 100% <100%> (ø) ⬆️
united_states_of_browsers/db_merge/db_merge.py 92.78% <76%> (-5.92%) ⬇️

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 5c91a76...e1c1c4d. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants