Skip to content

[gitlab] Add GitLab support for bot: show_config command - #390

Open
sondrebr wants to merge 6 commits into
EESSI:gitlabfrom
sondrebr:add-gitlab-support-bot-show-config-command
Open

[gitlab] Add GitLab support for bot: show_config command#390
sondrebr wants to merge 6 commits into
EESSI:gitlabfrom
sondrebr:add-gitlab-support-bot-show-config-command

Conversation

@sondrebr

Copy link
Copy Markdown
Collaborator

This PR adds GitLab support for the bot: show_config command. As part of this, the following changes were also made:

  • pull_request event handling added for GitLab:
    • pr removed as parameter of handle_pull_request_*_event() methods and deploy_built_artefacts()
    • Added event handler method handle_merge_request_event = handle_pull_request_event
    • handle_pull_request_opened_event() is properly implemented for GitLab
    • handle_pull_request_labeled_event() makes the bot post a "... not yet supported ..." message on GitLab when a bot:deploy label is added
    • handle_pull_request_closed_event() makes the bot post a "... not yet supported ..." message on GitLab (if the bot is configured to be chatty)
  • EventInfo changes:
    • pr_title property added (+ tests)
    • pr_number falls back to issue_number for issue_comment events on GitHub
  • README: GitLab disclaimer updated for show_config command support

The PR is ready for review, but it should only be merged after #387 has been merged.

Fall back to `GitHubEventInfo.issue_number` for `issue_comment` events
pull_request labeled and closed events are also "working" for GL
but not actually supported: For closed events, the bot responds
with a '... not yet supported' message; For labeled events,
the bot responds with a '... not yet supported' message if the
label is 'bot:deploy'.
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.

1 participant