Skip to main content

Browse the archive

Revision d497bb6a90dd3c9625c3c8e8f73278019e21983d authored by Roman Donchenko on 29 September 2023, 07:03:49 UTC, committed by GitHub on 29 September 2023, 07:03:49 UTC 
Remove the Git repository synchronization functionality (#6904)
 
This functionality has accumulated significant technical debt:

* Most importantly, it does not use the current authorization system,
  rendering it accessible only for admin users.

* It doesn't follow the regular API conventions and is not visible in
the API
  schema. This necessitates a special code in the SDK.

* The initialization code in `base.py` is not safe when multiple
instances of
the server starts at the same time (each instance may end up generating
its
  own key).

The team has decided that the cost of fixing these issues outweighs the benefit of the functionality, so remove it.
1 parent 4a487c3
Raw File
.coveragerc 
[run]
branch = true

source =
    cvat/apps/
    cvat-sdk/
    cvat-cli/
    utils/dataset_manifest

omit =
    cvat/settings/*
    */tests/*
    */test_*
    */_test_*
    */migrations/*

[report]
# Regexes for lines to exclude from consideration
exclude_lines =
    # Have to re-enable the standard pragma
    pragma: no cover

    # Don't complain about missing debug-only code:
    def __repr__
    if\s+[\w\.()]+\.isEnabledFor\(log\.DEBUG\):

    # Don't complain if tests don't hit defensive assertion code:
    raise AssertionError
    raise NotImplementedError

    # Don't complain if non-runnable code isn't run:
    if 0:
    if __name__ == .__main__.:

# don't fail on the code that can be found
ignore_errors = true

skip_empty = true
back to top