Skip to content

Do not change directory to run operation on cloned repository#219

Merged
myakove merged 1 commit intomainfrom
do-not-change-dir
Aug 5, 2023
Merged

Do not change directory to run operation on cloned repository#219
myakove merged 1 commit intomainfrom
do-not-change-dir

Conversation

@myakove
Copy link
Copy Markdown
Collaborator

@myakove myakove commented Aug 5, 2023

DOn't change directories in order to operation on cloned repository (such as tox, build container, etc.),
Once we run more than one check run, we fail since the directory was changed.

Done for all the except SonarQube (Need to find out how to do it)

@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

Report bugs in Issues

The following are automatically added:

  • Add reviewers from OWNER file (in the root of the repository) under reviewers section.
  • Set PR size label.
  • New issue is created for the PR. (Closed when PR is merged/closed)
  • Run pre-commit if .pre-commit-config.yaml exists in the repo.

Available user actions:

  • To mark PR as WIP comment /wip to the PR, To remove it from the PR comment /wip cancel to the PR.
  • To block merging of PR comment /hold, To un-block merging of PR comment /hold cancel.
  • To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
    verified label removed on each new commit push.
  • To cherry pick a merged PR comment /cherry-pick <target branch to cherry-pick to> in the PR.
    • Multiple target branches can be cherry-picked, separated by spaces. (/cherry-pick branch1 branch2)
    • Cherry-pick will be started when PR is merged
  • To re-run tox comment /retest tox in the PR.
  • To re-run build-container command /retest build-container in the PR.
  • To re-run python-module-install command /retest python-module-install in the PR.
  • To build and push container image command /build-and-push-container in the PR (tag will be the PR number).
  • To add a label by comment use /<label name>, to remove, use /<label name> cancel
Supported labels
  • hold
  • verified
  • wip
  • lgtm

@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/hold

@myakove myakove requested a review from rnetser August 5, 2023 14:54
@myakove myakove added the hold label Aug 5, 2023
@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/hold cancel

@myakove myakove removed the hold label Aug 5, 2023
@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/wip

@myakove myakove added the wip label Aug 5, 2023
@myakove myakove changed the title Do not change directory to run operation on cloned repository WIP: Do not change directory to run operation on cloned repository Aug 5, 2023
@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/retest tox

1 similar comment
@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/retest tox

@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/retest sonarqube

@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/retest build-container

@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/retest sonarqube

1 similar comment
@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/retest sonarqube

@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/retest build-container

@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/retest build-container tox build-container

@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/retest python-module-install

@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/retest build-container tox build-container python-module-install

@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/retest sonarqube

@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/retest tox

@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

/wip cancel

@myakove myakove removed the wip label Aug 5, 2023
@myakove myakove changed the title WIP: Do not change directory to run operation on cloned repository Do not change directory to run operation on cloned repository Aug 5, 2023
@myakove myakove merged commit adbc186 into main Aug 5, 2023
@myakove myakove deleted the do-not-change-dir branch August 5, 2023 15:46
@myakove
Copy link
Copy Markdown
Collaborator Author

myakove commented Aug 5, 2023

Container quay.io/myakove/github-webhook-server:latest pushed

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