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

fix: main process not running #144

Merged
merged 2 commits into from
Sep 4, 2023
Merged

fix: main process not running #144

merged 2 commits into from
Sep 4, 2023

Conversation

Ari-suhyeon
Copy link
Contributor

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • DevOps process (non-breaking change which improves efficiency and reliability for CI/CD)
  • Documentation only

What this PR does / why we need it:

  • metric process changed to run threads
  • update [ pre-push.sh > controller:test ] script

Which issue/s this PR fixes

fixes #143

How Has This Been Tested?

moon run tested - check metric data and main process logs

Checklist:

@Ari-suhyeon Ari-suhyeon added the type: bug type: Something isn't working label Sep 4, 2023
@Ari-suhyeon Ari-suhyeon added this to the v0.0.13 milestone Sep 4, 2023
@Ari-suhyeon Ari-suhyeon requested a review from pueding September 4, 2023 04:24
@Ari-suhyeon Ari-suhyeon linked an issue Sep 4, 2023 that may be closed by this pull request
main process not running. #143
Closed
@pueding pueding merged commit e56628a into main Sep 4, 2023
@pueding pueding deleted the 143-main-process-not-running branch September 4, 2023 07:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@pueding pueding pueding approved these changes

Assignees
No one assigned
Labels
type: bug type: Something isn't working
Projects
None yet
Milestone
v0.0.13
Development

Successfully merging this pull request may close these issues.

main process not running.
2 participants
@Ari-suhyeon @pueding