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.

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

Modify lazy_static to LazyLock #134

Merged
merged 1 commit into from
Aug 5, 2024
Merged

Modify lazy_static to LazyLock #134

merged 1 commit into from
Aug 5, 2024

Conversation

henry0715-dev
Copy link
Contributor

Desc

Use LazyLock of rust std library
Remove lazy_static dependency

Reference

#133

Use LazyLock of rust std library
Remove lazy_static dependency
@henry0715-dev
Copy link
Contributor Author

giganto-client에서 lazy_static은 test 코드에서만 사용되는 것으로 파악
LazyLock으로 변경시에 실제 수행되는 코드에서는 영향도가 없는 것으로 생각 됨

@sehkone
Copy link
Contributor

sehkone commented Aug 5, 2024

@henry0715-dev
According to our rules for communication, the number 11, the only repo owner can assign assignees to PRs for the main branch. Since Sophie is not available this week, I assigned this to @kimhanbeom.

@sehkone sehkone assigned kimhanbeom and unassigned sophie-cluml Aug 5, 2024
@sehkone sehkone requested review from kimhanbeom and removed request for sophie-cluml August 5, 2024 02:26
@sehkone
Copy link
Contributor

sehkone commented Aug 5, 2024

@henry0715-dev
Unlike PR assignees, assigning PR reviewers is not restricted.

@henry0715-dev
Copy link
Contributor Author

Reviewer 변경 감사합니다.

@sehkone
Copy link
Contributor

sehkone commented Aug 5, 2024

@henry0715-dev

제가 @kimhanbeom으로 PR assignee를 변경한 것이었습니다.

PR을 올리는 사람이 reviewer는 맘대로 지정해도 되는데, assignee는 repository owner가 지정해야 한다는 것이 규칙인데요, 즉, @henry0715-dev가 reviewer만 @sophie-cluml로 지정하는 것이 맞다는 것입니다.

@henry0715-dev가 assignee를 @sophie-cluml로 지정하셨는데, 저는 repository owner는 아니나 권한이 있는 자로서 @kimhanbeom으로 변경을 한 것입니다.

@henry0715-dev가 reviewer를 @sophie-cluml로 지정한 것은 규칙에 어긋나지 않지만 당장 자리에 없으므로 제가 reviewer로 @kimhanbeom으로 변경한 것입니다.

제가 다시 assignee는 @kimhanbeom으로 지정하겠습니다.

@sehkone
Copy link
Contributor

sehkone commented Aug 5, 2024

@henry0715-dev
그리고, 이 repo owner는 @kimhanbeom이니 @kimhanbeom 또는 권한있는 사람이 assignee를 다른 사람으로 지정하지 않는다면 자동으로 assignee는 @kimhanbeom이 됩니다.

@kimhanbeom kimhanbeom merged commit 17fd4ae into main Aug 5, 2024
8 checks passed
@kimhanbeom kimhanbeom deleted the henry/replace-lazylock branch August 5, 2024 04:13
@sehkone
Copy link
Contributor

sehkone commented Aug 5, 2024

@henry0715-dev

조금 더 부연하면,

  • 모든 repositories에는 main 브랜치의 경우 owner와 deputy가 지정되어 있습니다.
  • 올라온 PR 진행은 owner가 담당합니다. 즉 assignee가 자동적으로 owner인 것입니다. owner 대신 deputy가 담당할 수도 있는데, 이 결정은 owner 이상의 권한이 있는 인원만 할 수 있습니다.
  • 상기 이유로 PR 올리는 사람은 assignee를 지정하지 않는 것입니다.
  • Reviewer는 PR 올리는 사람이 임의로 지정할 수 있지만, owner 이상의 권한이 있는 인원이 이를 변경할 수 있습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants