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 #631

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

CORTX-30785: Fixing the Alex bug #631

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'"

Design

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

Coding

Checklist for Author

  • Coding conventions are followed and code is consistent

Testing

Checklist for Author

  • Unit and System Tests are added
  • Test Cases cover Happy Path, Non-Happy Path and Scalability
  • Testing was performed with RPM

Impact Analysis

Checklist for Author/Reviewer/GateKeeper

  • Interface change (if any) are documented
  • Side effects on other features (deployment/upgrade)
  • Dependencies on other component(s)

Review Checklist

Checklist for Author

  • JIRA number/GitHub Issue added to PR
  • PR is self reviewed
  • Jira and state/status is updated and JIRA is updated with PR link
  • Check if the description is clear and explained

Documentation

Checklist for Author

  • Changes done to WIKI / Confluence page / Quick Start Guide

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]>
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.

1 participant