pijul_org / pijul

#276 pijul add .pijul/hooks/pre-record works

Opened by lthms, on May 13, 2018
Bug
Closed
Closed
Closed
lthms commented on May 13, 2018

I tried, just to see if that would work. Turns out it did. I am not sure this is a good behaviour.

I didn't tried to record the patch, but I can see how troublesome it can become if I try to record, for instance, .pijul/pristin/db.

yory commented on May 14, 2018

Yes, git refuses to add paths in its .git folder. We should too. When we want to version hooks, the best practice is putting them in a hooks folder in the root of the repo, then symlink them into .pijul/hooks. We don't want to interfere with an other user's hooks.

lthms commented on May 14, 2018

This is a good opportunity to fix pijul add with regards to ignored files… or to deprecate it in favour of pijul record -n, which is getting it right as far as I know.

Having -n to become the default behaviour would be great, especially as now we have the possibility to add the file to .ignore or .pijul/local/ignore from pijul record.

pmeunier commented on February 27, 2019

I just fixed this! Files containing a .pijul component cannot be added to the repository anymore.

pmeunier closed this discussion on February 27, 2019
pmeunier commented on February 27, 2019

I just fixed this! Files containing a .pijul component cannot be added to the repository anymore.

pmeunier closed this discussion on February 27, 2019
pmeunier commented on February 27, 2019

I just fixed this! Files containing a .pijul component cannot be added to the repository anymore.

pmeunier closed this discussion on February 27, 2019