Code

analytics: refactor into a module#2826

Code

Merged

efiop merged 46 commits intomasteriterative/dvc:masterfrom unknown repositoryCopy head branch name to clipboardDec 9, 2019

+253-277

Commits

Commits on Nov 20, 2019

• analytics: refactor into a module

  

  Mr. Outis

  committed
  ef1f038

Commits on Nov 26, 2019

• analytics: return OS when system info not supported

  

  Mr. Outis

  committed
  b618d03

• analytics: collect system info under the same module

  

  Mr. Outis

  committed
  7d3cbd7

• analytics: move user_id module to a method

  Show description for 64d102c
  

  Mr. Outis

  committed
  64d102c

• 💅 sort methods

  

  Mr. Outis

  committed
  2e7a6d3

• analytics: move to single file

  Show description for 5cb437b
  

  Mr. Outis

  committed
  5cb437b

• analytics: go back to daemon implementation

  Show description for 610d1ee
  

  Mr. Outis

  committed
  610d1ee

• analytics: exit_code -> return_code

  Show description for d958870
  

  Mr. Outis

  committed
  d958870

• :face_palm: fix wrong module

  

  Mr. Outis

  committed
  34d3004

• tests: analytics.find_or_create_user_id

  

  Mr. Outis

  committed
  5261ed7

Commits on Nov 27, 2019

• py2: FileNotFoundError -> IOError

  

  Mr. Outis

  committed
  467b110

• 💅 change naming and docstring

  

  Mr. Outis

  committed
  48b4cba

• 💅 black

  

  Mr. Outis

  committed
  6bc2403

• tests: functional and unit tests for analytics

  

  Mr. Outis

  committed
  f995204

• 💅 deepsource

  

  Mr. Outis

  committed
  2a5cac5

• 💅 sort imports

  

  Mr. Outis

  committed
  e9d56a9

• tests: set temporary global config

  

  Mr. Outis

  committed
  787ea9c

Commits on Nov 28, 2019

• py2 compat issues 🤷

  

  Mr. Outis

  committed
  e69fffa

• 💅 correct wording "disenabled" -> "disabled"

  

  Mr. Outis

  committed
  656f986

Commits on Dec 2, 2019

• analytics: send report without loading it to memory

  Show description for 946dfc1
  

  Mr. Outis

  committed
  946dfc1

• analytics: document why tmp_global_config is needed

  

  Mr. Outis

  committed
  5596015

Commits on Dec 3, 2019

• analytics: use fspath instead of str

  

  Mr. Outis

  committed
  2c8e149

• analytics: define report schema and use it on tests

  

  Mr. Outis

  committed
  ca485db

• 💅 formatting

  

  Mr. Outis

  committed
  e0d47a1

• analytics: move report schema to tests

  

  Mr. Outis

  committed
  8b5ca25

• analytics: collect and send on daemon

  

  Mr. Outis

  committed
  8a32b28

• 💅 more specific comment about analytics

  

  Mr. Outis

  committed
  624cbb2

• tests: mock analytics.collect while testing daemon

  

  Mr. Outis

  committed
  285620d

Commits on Dec 4, 2019

• py35: support for fspath(pathlib.Path())

  

  Mr. Outis

  committed
  02eaa9a

Commits on Dec 5, 2019

• py2: use convert_to_unicode instead of str

  

  Mr. Outis

  committed
  ede0103

• tests: add unit test for analytics.system_info

  

  Mr. Outis

  committed
  7a3aae1

• tests: isolate global config from analytics tests

  Show description for 05a6868
  

  Mr. Outis

  committed
  05a6868

• analytics: use a tempfile for inter-process communication

  

  Mr. Outis

  committed
  71ead3f

Commits on Dec 6, 2019

• remove pathlib / fspath changes related to the patch

  

  Mr. Outis

  committed
  7e8bdbd

• tests: adjust scm_class schema

  

  Mr. Outis

  committed
  bc2471c

• compat: bring back unicode literals

  

  Mr. Outis

  committed
  c82d9dc

• tests: stringify tmp_global_config since it doesnt return a pathlike object

  

  Mr. Outis

  committed
  f704e54

• analytics: remove the report after sending it

  

  Mr. Outis

  committed
  da064e2

• tests: use str, builtin_str in schema

  

  Mr. Outis

  committed
  5a80334

• analytics: define private methods

  

  Mr. Outis

  committed
  2145daa

Commits on Dec 8, 2019

• analytics: collect execution info only when available

  

  Mr. Outis

  committed
  c50bd17

• analytics: raise error when collecting a not supported os

  

  Mr. Outis

  committed
  146e75b

• analytics: AttributeError -> KeyError

  

  Mr. Outis

  committed
  c9f958d

• 💅 add dot to the end of the comment

  

  Mr. Outis

  committed
  3244f10

• tests: require keys on analytics report schema

  

  Mr. Outis

  committed
  6e4c3d6

• 💅 black

  

  Mr. Outis

  committed
  5dd6300