Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed nested git repositories when running setup script #79

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

Conversation

kartik-212004
Copy link

@kartik-212004 kartik-212004 commented Dec 17, 2024

Issue

The current script clones a repository inside the main project directory (EduAid), which introduces a nested .git folder.
This leads to the following problems:

  1. Running git status shows unexpected changes from the nested repository.
  2. Executing git add . causes conflicts due to the presence of multiple .git folders.
  3. It complicates version control within the main project.

Fixes Issue : #76

Solution

To resolve the issue, the script has been updated to remove the .git directory from the cloned repository immediately after cloning.

Changes Made

  • Added a command to delete the .git folder within the cloned repository:
    rm -rf $REPO_DIR/.git

@manthanabc
Copy link

manthanabc commented Dec 18, 2024

To resolve the issue, the script has been updated to remove the .git directory from the cloned repository immediately after cloning.

I fail to understand why your script is still cloning the entire repository again in the backend folder, and removing it from git's tracking?, as far as i can tell there is no need for this folder's existence in first place!

@kartik-212004
Copy link
Author

kartik-212004 commented Dec 18, 2024

It was cloning with the previous one too , i just added the functionality to remove the .git folder so that one could commit changes

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.

2 participants