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

ci: add check for PR author organization membership and assign them #5521

Merged
merged 4 commits into from
Jan 14, 2025

Conversation

WANZARGEN
Copy link
Member

To Reviewers

  • Minor change, self-reviewed (style, chore, ci, test, docs)
  • Previously reviewed in feature branch, no further review needed
  • Self-merge allowed for solo developers or urgent changes

Description (optional)

Things to Talk About (optional)

Signed-off-by: Wanjin Noh <[email protected]>
Copy link

vercel bot commented Jan 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
console ⬜️ Ignored (Inspect) Visit Preview Jan 14, 2025 1:11pm
dashboard ⬜️ Ignored (Inspect) Visit Preview Jan 14, 2025 1:11pm

Copy link
Contributor

🎉 @piggggggggy and @kkdy21 have been randomly selected as the reviewers! Please review. 🙏

@github-actions github-actions bot added the pending/review Pull Request is awaiting review. No reviewers have started reviewing yet. label Jan 14, 2025
@WANZARGEN WANZARGEN closed this Jan 14, 2025
@WANZARGEN WANZARGEN reopened this Jan 14, 2025
@github-actions github-actions bot removed the pending/review Pull Request is awaiting review. No reviewers have started reviewing yet. label Jan 14, 2025
@WANZARGEN WANZARGEN changed the title chore: for ci test ci: add check for PR author organization membership and assign them Jan 14, 2025
@WANZARGEN WANZARGEN merged commit 62686a9 into cloudforet-io:develop Jan 14, 2025
5 of 6 checks passed
WANZARGEN added a commit to WANZARGEN/console that referenced this pull request Jan 16, 2025
…loudforet-io#5521)

* chore: for ci test

Signed-off-by: Wanjin Noh <[email protected]>

* ci: add check for PR author organization membership and assign them

Signed-off-by: Wanjin Noh <[email protected]>

* ci: update pull request review labeler to handle reopened events

Signed-off-by: Wanjin Noh <[email protected]>

* ci: enhance pull request review labeler to handle missing review state

Signed-off-by: Wanjin Noh <[email protected]>

---------

Signed-off-by: Wanjin Noh <[email protected]>
skdud4659 added a commit that referenced this pull request Jan 16, 2025
* ci: add check for PR author organization membership and assign them (#5521)

* chore: for ci test

Signed-off-by: Wanjin Noh <[email protected]>

* ci: add check for PR author organization membership and assign them

Signed-off-by: Wanjin Noh <[email protected]>

* ci: update pull request review labeler to handle reopened events

Signed-off-by: Wanjin Noh <[email protected]>

* ci: enhance pull request review labeler to handle missing review state

Signed-off-by: Wanjin Noh <[email protected]>

---------

Signed-off-by: Wanjin Noh <[email protected]>

* chore: ci test (#5522)

Signed-off-by: Wanjin Noh <[email protected]>

* fix(reference-store): correct logic for selecting fetcher based on version (#5536)

- Inverted the condition for determining the collector and region fetchers
- Ensured correct API client is used based on alert manager version

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

* chore: changed css (#5537)

Signed-off-by: NaYeong,Kim <[email protected]>

---------

Signed-off-by: Wanjin Noh <[email protected]>
Signed-off-by: sulmo <[email protected]>
Signed-off-by: NaYeong,Kim <[email protected]>
Co-authored-by: WANJIN <[email protected]>
Co-authored-by: sulmo <[email protected]>
@skdud4659 skdud4659 mentioned this pull request Jan 16, 2025
3 tasks
WANZARGEN added a commit to WANZARGEN/console that referenced this pull request Jan 16, 2025
…loudforet-io#5521)

* chore: for ci test

Signed-off-by: Wanjin Noh <[email protected]>

* ci: add check for PR author organization membership and assign them

Signed-off-by: Wanjin Noh <[email protected]>

* ci: update pull request review labeler to handle reopened events

Signed-off-by: Wanjin Noh <[email protected]>

* ci: enhance pull request review labeler to handle missing review state

Signed-off-by: Wanjin Noh <[email protected]>

---------

Signed-off-by: Wanjin Noh <[email protected]>
WANZARGEN added a commit that referenced this pull request Jan 19, 2025
* ci: add check for PR author organization membership and assign them (#5521)

* chore: for ci test

Signed-off-by: Wanjin Noh <[email protected]>

* ci: add check for PR author organization membership and assign them

Signed-off-by: Wanjin Noh <[email protected]>

* ci: update pull request review labeler to handle reopened events

Signed-off-by: Wanjin Noh <[email protected]>

* ci: enhance pull request review labeler to handle missing review state

Signed-off-by: Wanjin Noh <[email protected]>

---------

Signed-off-by: Wanjin Noh <[email protected]>

* chore: ci test (#5522)

Signed-off-by: Wanjin Noh <[email protected]>

* docs: update pr template

Signed-off-by: Wanjin Noh <[email protected]>

* docs: update pr template

Signed-off-by: Wanjin Noh <[email protected]>

* refactor: update pull_request_random_reviewer_assignment.yaml

Signed-off-by: Wanjin Noh <[email protected]>

* ci: update labeling workflows

Signed-off-by: Wanjin Noh <[email protected]>

* chore: ci test self_approved/review label (#11)

* refactor: remove unnecessary code in pull_request_review_state_labeler.yaml

Signed-off-by: Wanjin Noh <[email protected]>

* refactor: update pull_request_random_reviewer_assignment.yaml

Signed-off-by: Wanjin Noh <[email protected]>

---------

Signed-off-by: Wanjin Noh <[email protected]>

* fix: add console log for review state in pull_request_review_state_labeler.yaml

Signed-off-by: Wanjin Noh <[email protected]>

* fix: add corresponding review state labels

Signed-off-by: Wanjin Noh <[email protected]>

* fix: enhance label removal logic in pull_request_review_state_labeler.yaml

Signed-off-by: Wanjin Noh <[email protected]>

* fix: update label removal method in pull_request_review_state_labeler.yaml

Signed-off-by: Wanjin Noh <[email protected]>

* fix: refactor label removal process to remove labels individually

Signed-off-by: Wanjin Noh <[email protected]>

* fix: streamline label removal logic in to enhance clarity and maintainability

Signed-off-by: Wanjin Noh <[email protected]>

---------

Signed-off-by: Wanjin Noh <[email protected]>
skdud4659 added a commit that referenced this pull request Jan 20, 2025
* ci: add check for PR author organization membership and assign them (#5521)

* chore: for ci test

Signed-off-by: Wanjin Noh <[email protected]>

* ci: add check for PR author organization membership and assign them

Signed-off-by: Wanjin Noh <[email protected]>

* ci: update pull request review labeler to handle reopened events

Signed-off-by: Wanjin Noh <[email protected]>

* ci: enhance pull request review labeler to handle missing review state

Signed-off-by: Wanjin Noh <[email protected]>

---------

Signed-off-by: Wanjin Noh <[email protected]>

* chore: ci test (#5522)

Signed-off-by: Wanjin Noh <[email protected]>

* fix(reference-store): correct logic for selecting fetcher based on version (#5536)

- Inverted the condition for determining the collector and region fetchers
- Ensured correct API client is used based on alert manager version

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

* chore: changed css (#5537)

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: apply qa and additional requests (#5552)

* chore: changed icon color

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: create alert creation button at service page

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: update history error modal

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: create app reference store and apply app name at timeline

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: apply resource link at alert detail page

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: apply 'user group' text at member modal

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: apply raw data modal at webhook error tab

Signed-off-by: NaYeong,Kim <[email protected]>

* fix: fixed timeline bug

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: apply filter retention

Signed-off-by: NaYeong,Kim <[email protected]>

* chore: changed translations and style

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: changed create alert button position

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: fixed condition form bug and translate operator

Signed-off-by: NaYeong,Kim <[email protected]>

* chore: translations

Signed-off-by: NaYeong,Kim <[email protected]>

---------

Signed-off-by: NaYeong,Kim <[email protected]>

* fix: fixed qa bug (#5554)

* fix: fixed minor bug (timeline)

Signed-off-by: NaYeong,Kim <[email protected]>

* fix: fixed qa bug

Signed-off-by: NaYeong,Kim <[email protected]>

---------

Signed-off-by: NaYeong,Kim <[email protected]>

* feat: apply sort (#5555)

Signed-off-by: NaYeong,Kim <[email protected]>

---------

Signed-off-by: Wanjin Noh <[email protected]>
Signed-off-by: sulmo <[email protected]>
Signed-off-by: NaYeong,Kim <[email protected]>
Co-authored-by: WANJIN <[email protected]>
Co-authored-by: sulmo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant