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

Bad diff highlight #20152

Closed
Fogapod opened this issue Jun 27, 2022 · 2 comments · Fixed by #19958
Closed

Bad diff highlight #20152

Fogapod opened this issue Jun 27, 2022 · 2 comments · Fixed by #19958
Labels

Comments

@Fogapod
Copy link
Contributor

Fogapod commented Jun 27, 2022

Description

my changes:

diff --git a/core/deploy_core.sh b/core/deploy_core.sh
index ea37829..cec1cc4 100644
--- a/core/deploy_core.sh
+++ b/core/deploy_core.sh
@@ -57,7 +57,7 @@ echo "Complete. Start deploy process."
 
 PROJECT_DIR=$BASE_PATH"/$PROJECT_DIR_NAME"
 PROJECT_DEPLOY_DIR=$PROJECT_DIR"_deploy"
-FILES_FOR_DEL=("Pipfile" "Pipfile.lock" ".git" ".gitignore" ".gitmodules" "README.md" "config.sample.ini" "runserver.sh" "runserver_pipenv.sh" "supervisord.conf" "supervisord.pipenv.conf" "systemd.sample" "requirements.txt" "deploy.sh")
+FILES_FOR_DEL=(".git" ".gitignore" "README.md" "config.sample.ini" "runserver.sh" "runserver_pipenv.sh" "supervisord.conf" "supervisord.pipenv.conf" "systemd.sample" "requirements.in" "requirements.txt" "deploy.sh")
 SUPERVISORCTL_CMD='supervisorctl -c supervisord.conf '
 
 ################################# DEPLOY SCRIPT ########################################

how gitea displays this file:
image

Pipfile" "Pipfile.lock part got removed but only Pipfile" is red

Gitea Version

1.17.0+rc1

Can you reproduce the bug on the Gitea demo site?

No

Log Gist

No response

Screenshots

No response

Git Version

2.36.1

Operating System

No response

How are you running Gitea?

Locally compiled binary

Database

PostgreSQL

@wxiaoguang
Copy link
Contributor

@Fogapod
Copy link
Contributor Author

Fogapod commented Jun 27, 2022

For some reason i thought that is already merged. Thanks

@go-gitea go-gitea locked and limited conversation to collaborators May 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants