Skip to content
This repository has been archived by the owner on Jun 7, 2023. It is now read-only.

github bot initial iteration #10

Merged
merged 1 commit into from
Jul 8, 2021
Merged

Conversation

KPostOffice
Copy link
Contributor

Initial bot implementation. A simple cli program which can run as part of a workflow and relies on an underlying library function.

@sesheta sesheta requested review from goern and harshad16 June 29, 2021 17:36
@sesheta sesheta added the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label Jun 29, 2021
@goern
Copy link
Member

goern commented Jun 29, 2021

/lgtm

@sesheta sesheta added the lgtm Indicates that a PR is ready to be merged. label Jun 29, 2021
@sesheta sesheta removed the lgtm Indicates that a PR is ready to be merged. label Jun 29, 2021
@KPostOffice
Copy link
Contributor Author

/retest

@sesheta
Copy link

sesheta commented Jun 29, 2021

Hi @KPostOffice,
you are not authorised to run /retest command.
Please contact a MEMBER.

@goern
Copy link
Member

goern commented Jun 30, 2021

/test all

@KPostOffice
Copy link
Contributor Author

@goern Can you please try again?

@goern
Copy link
Member

goern commented Jul 1, 2021

@KPostOffice you can also try it ;)

/test all

Copy link
Member

@harshad16 harshad16 left a comment

Choose a reason for hiding this comment

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

/lgtm
good work 💯

"""Templates for issues and PRs opened by Varangian git application."""

__name__ = "varangian_git_app"
__version__ = "0.1.0"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
__version__ = "0.1.0"
__version__ = "0.0.0"

As this project is not yet released , it should have any version.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

++ thanks harshad

@sesheta sesheta added the lgtm Indicates that a PR is ready to be merged. label Jul 1, 2021
@sesheta sesheta removed the lgtm Indicates that a PR is ready to be merged. label Jul 1, 2021
Copy link
Member

@harshad16 harshad16 left a comment

Choose a reason for hiding this comment

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

/lgtm
thanks 💯

@sesheta sesheta added the lgtm Indicates that a PR is ready to be merged. label Jul 6, 2021
@harshad16 harshad16 assigned KPostOffice and unassigned harshad16 Jul 6, 2021
@goern goern mentioned this pull request Jul 6, 2021
Closed
@goern
Copy link
Member

goern commented Jul 8, 2021

/approve

@sesheta
Copy link

sesheta commented Jul 8, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: goern

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sesheta sesheta added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jul 8, 2021
@sesheta sesheta merged commit 3781802 into AICoE:main Jul 8, 2021
@KPostOffice KPostOffice deleted the bot-iteration-0 branch December 1, 2021 20:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants