Skip to content

Add FILES env variable #114

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

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

Conversation

Simx303
Copy link

@Simx303 Simx303 commented Sep 23, 2024

FILES environment variable will be available for scripts by adding $FILES to their script.
It will list all staged files for commit.

Here is an example:

STAGED
---
coolapp/
├─ .editorconfig +
├─ README.md
├─ src/
│  ├─ main.js +
│  ├─ utils.js +


> echo $FILES
.editorconfig src/main.js src/utils.js

> npx do-something $FILES
...

echo "$FILES" | xargs git add at the end will add again the changed files for commit.

`FILES` environment variable will be available for scripts by adding `... $FILES ...` to their script.

It will list all staged files for commit.
@toplenboren
Copy link
Owner

Hello, thanks for the PR 👍

From what I understand, you are adding functionality to automatically stage all files whenever a git hook is set

I don't think this should be enabled by default because one can have staged scratches and scripts that he/she donesnt want to include in my repository

Can you please share your usecase? Why not just create a git hook to stage all files?

@JounQin
Copy link
Collaborator

JounQin commented Apr 7, 2025

I believe it could break Windows compatibility?

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.

3 participants