Skip to content
This repository has been archived by the owner on May 15, 2024. It is now read-only.

feat: add cleanup test, and isolate id mapping code and cleanup code #218

Merged
merged 8 commits into from
Nov 11, 2023
Merged

feat: add cleanup test, and isolate id mapping code and cleanup code #218

merged 8 commits into from
Nov 11, 2023

Commits on Nov 10, 2023

  1. refactor: separate out cleanup scheduler and id manager

    @elijaharita
    elijaharita committed Nov 10, 2023
    Configuration menu
    Copy the full SHA
    018be8a View commit details
    Browse the repository at this point in the history

  2. feat: cleanup test

    @elijaharita
    elijaharita committed Nov 10, 2023
    Configuration menu
    Copy the full SHA
    92d6a6a View commit details
    Browse the repository at this point in the history

  3. fix: use test temp dir instead of system

    @elijaharita
    elijaharita committed Nov 10, 2023
    Configuration menu
    Copy the full SHA
    68421c6 View commit details
    Browse the repository at this point in the history

  4. fix: cleanup test race

    @elijaharita
    elijaharita committed Nov 10, 2023
    Configuration menu
    Copy the full SHA
    26f6be4 View commit details
    Browse the repository at this point in the history

  5. fix: comment unused function idMap.remove() to suppress warning

    @elijaharita
    elijaharita committed Nov 10, 2023
    Configuration menu
    Copy the full SHA
    664821e View commit details
    Browse the repository at this point in the history

  6. fix: address comments

    @elijaharita
    elijaharita committed Nov 10, 2023
    Configuration menu
    Copy the full SHA
    6229fca View commit details
    Browse the repository at this point in the history

  7. fix: final fixes

    @elijaharita
    elijaharita committed Nov 10, 2023
    Configuration menu
    Copy the full SHA
    d6193a9 View commit details
    Browse the repository at this point in the history

  8. refactor: move defer ticker.stop() to right after ticker init

    @elijaharita
    elijaharita committed Nov 10, 2023
    Configuration menu
    Copy the full SHA
    7adcc08 View commit details
    Browse the repository at this point in the history