Skip to content

Commit

Permalink
fix(telemetry): Let environment variables override consent from .tele…
Browse files Browse the repository at this point in the history
…metry file (#791)

* Always set self._consent

Signed-off-by: Laura Couto <[email protected]>

* Lint

Signed-off-by: Laura Couto <[email protected]>

* Fix merge conflict

Signed-off-by: Laura Couto <[email protected]>

---------

Signed-off-by: Laura Couto <[email protected]>
Signed-off-by: L. R. Couto <[email protected]>
Co-authored-by: ElenaKhaustova <[email protected]>
Co-authored-by: Juan Luis Cano Rodríguez <[email protected]>
  • Loading branch information
3 people authored Aug 1, 2024
1 parent 84cc29a commit c2ef4c5
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions kedro-telemetry/kedro_telemetry/plugin.py
Original file line number Diff line number Diff line change
Expand Up @@ -202,8 +202,7 @@ def after_command_run(self):
def after_context_created(self, context):
"""Hook implementation to send project statistics data to Heap"""

if self._consent is None:
self._consent = _check_for_telemetry_consent(context.project_path)
self._consent = _check_for_telemetry_consent(context.project_path)
self._project_path = context.project_path

@hook_impl
Expand Down

0 comments on commit c2ef4c5

Please sign in to comment.