Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[ASSET-51] Remove CI pipeline on public repo #57

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[ASSET-51] Remove CI pipeline on public repo #57

wants to merge 1 commit into from
+6 −31

Conversation

jormal
Copy link
Collaborator

@jormal jormal commented Jan 23, 2025 •
edited
Loading

Pull Request

Description

불필요해진 CI pipeline을 제거합니다.

Related issue: ASSET-51

Changes

Types of Changes (multiple options can be selected)

  • Create asset information
  • Update asset information
  • Delete asset information
  • Other Settings

Checklist

  • Did you pass the tests? (Did you execute pytest -l -all in your PC and get a green light?)
  • Did you run the pre-process?
  • Have you added and run tests to validate the changes?
  • Have you updated the documentation?
  • Have you updated the version in {PWD}/VERSION if you need?

@jormal
[ASSET-50] Log public IP when RPC Test failure (#56)
f1f9994 
* ASSET-50 Add public IP log on RPC failure

Signed-off-by: jormal <[email protected]>

* ASSET-50 Update CI test to verbose print

Signed-off-by: jormal <[email protected]>

* ASSET-50 Fix the action runs on self-hosted machine

Signed-off-by: jormal <[email protected]>

---------

Signed-off-by: jormal <[email protected]>
@jormal jormal added the settings Set this repository label Jan 23, 2025
@jormal jormal self-assigned this Jan 23, 2025
@jormal jormal requested a review from a team as a code owner January 23, 2025 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zibersaioros zibersaioros zibersaioros approved these changes

@gihyeonsung gihyeonsung Awaiting requested review from gihyeonsung

@luke-yoon luke-yoon Awaiting requested review from luke-yoon

@oh51dy oh51dy Awaiting requested review from oh51dy

@yjkellyjoo yjkellyjoo Awaiting requested review from yjkellyjoo

Assignees

@jormal jormal

Labels
settings Set this repository
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jormal @zibersaioros