Skip to content
This repository has been archived by the owner on May 3, 2024. It is now read-only.

CORTX-30785: Fixing the Alex bug #537

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

CORTX-30785: Fixing the Alex bug #537

wants to merge 1 commit into from

Conversation

venkuppu-chn
Copy link
Contributor

@venkuppu-chn venkuppu-chn commented May 6, 2022

Signed-off-by: Venkatesh K [email protected]

Problem Statement:

Alex's Github Action is fails with an error "reviewdog: PullRequest needs 'git' command: failed to run 'git rev-parse --show-prefix'"

Solution Design:

Changed the permissions of the directory where the target repository is checkeout in the workflow of GitHub Actions.

Coding:

  • Did you follow coding standard and verified it with flake8?
  • Did your address all Codacy issues?

Testing:

  • Unittest updated and all unittests are passing?
  • Sanity/Self tests are updated (if applicable)?
  • Manual testing done covering happy path and non-happy path?

Integration:

  • If there is any interface change, did you communicate it to other components?
  • If there is any interface change, other componenent gate gatekeepers are ready to accept the change?
  • Did you complete deployment test if applicable?

PR checklist:

  • Did you add Jira number to the PR?
  • DCO and cla-signed?

Problem: 
Alex's Github Action is fails with an error "reviewdog: PullRequest needs 'git' command: failed to run 'git rev-parse --show-prefix'" 

Solution: 
Changed the permissions of the directory where the target repository is checkeout in the workflow of GitHub Actions.

Signed-off-by: Venkatesh K <[email protected]>
@cdeshmukh cdeshmukh requested a review from sachitanands May 6, 2022 08:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants