Skip to content

Add a check for actor authorization#8

Merged
soehms merged 1 commit intosync_labelsfrom
sync_labels_actor_authorized
Apr 10, 2023
Merged

Add a check for actor authorization#8
soehms merged 1 commit intosync_labelsfrom
sync_labels_actor_authorized

Conversation

@soehms
Copy link
Copy Markdown
Owner

@soehms soehms commented Apr 10, 2023 •
edited
Loading

📚 Description

In this PR I

  • add a new method: actor_authorized which is used before s: positive review is set automatically
  • add a flag SYNC_LABELS_ACTIVE which must be set as variable in the repository to activate the bot
  • add a flag SYNC_LABELS_LOG_LEVEL which can be set as variable in the repository to switch the log level
  • add a scheduler in the workflow for cleaning the warning comments before the time_frame of clean_warnings runs out

📝 Checklist

  • The title is concise, informative, and self-explanatory.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation accordingly.

⌛ Dependencies

@gitpod-io
Copy link
Copy Markdown

gitpod-io bot commented Apr 10, 2023

@soehms
Copy link
Copy Markdown
Owner Author

soehms commented Apr 10, 2023

Starting tests for sync_labels

@soehms
Copy link
Copy Markdown
Owner Author

soehms commented Apr 10, 2023

Test action opened

@soehms
Copy link
Copy Markdown
Owner Author

soehms commented Apr 10, 2023

Test action reopened

@soehms
Copy link
Copy Markdown
Owner Author

soehms commented Apr 10, 2023

Test action closed

@soehms
Copy link
Copy Markdown
Owner Author

soehms commented Apr 10, 2023

Test action labeled

@soehms
Copy link
Copy Markdown
Owner Author

soehms commented Apr 10, 2023

Label Sync Warning: Label s: positive review can not be added. Please use the GitHub review functionality

@soehms
Copy link
Copy Markdown
Owner Author

soehms commented Apr 10, 2023

Test action ready_for_review

@soehms
Copy link
Copy Markdown
Owner Author

soehms commented Apr 10, 2023

Test action synchronize

@soehms
Copy link
Copy Markdown
Owner Author

soehms commented Apr 10, 2023

Test action review_requested

@soehms
Copy link
Copy Markdown
Owner Author

soehms commented Apr 10, 2023

Test action converted_to_draft

@soehms
Copy link
Copy Markdown
Owner Author

soehms commented Apr 10, 2023

Test action submitted

soehms
soehms commented Apr 10, 2023
View reviewed changes
Copy link
Copy Markdown
Owner Author

@soehms soehms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s: positive review

soehms
soehms commented Apr 10, 2023
View reviewed changes
Copy link
Copy Markdown
Owner Author

@soehms soehms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s: needs work

soehms
soehms commented Apr 10, 2023
View reviewed changes
Copy link
Copy Markdown
Owner Author

@soehms soehms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s: needs review

@soehms
Copy link
Copy Markdown
Owner Author

soehms commented Apr 10, 2023

Label Sync Warning: Label s: needs review can not be added, since there are unresolved reviews

soehms
soehms commented Apr 10, 2023
View reviewed changes
Copy link
Copy Markdown
Owner Author

@soehms soehms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s: needs info

@soehms soehms removed the r: worksforme Works for me label Apr 10, 2023
@soehms soehms merged commit aa449ef into sync_labels Apr 10, 2023
@soehms soehms mentioned this pull request Apr 10, 2023
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@github-actions github-actions[bot] github-actions[bot] requested changes

Assignees

No one assigned

Labels

p: minor /4 s: needs review s: positive review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@soehms