[FIX] Exclude frozen.yaml from pre-commit because it is generated automatically

[florian-dacosta]

If we delete it and re-create it, it is full of unwanted diff because of pre-commit reformat

@bealdav @Kev-Roche @sebastienbeau @hparfr

[bealdav]

Thnaks a lot

[hparfr]

is it may be better to only include local-src ?

@sebastienbeau

[florian-dacosta]

is it may be better to only include local-src ?

@sebastienbeau

It makes sense for me. (If we still can exclude some stuff inside local-src)

[hparfr]

It makes sense for me. (If we still can exclude some stuff inside local-src)

can you give me some examples ?

[hparfr]

you need also to update this file: update_oca_file.py

[florian-dacosta]

It makes sense for me. (If we still can exclude some stuff inside local-src)

can you give me some examples ?

Well, I am not sure, but I think I had problem already, for instance with CSV files used for some tests in any module.
If a module import a csv file from an external source, I may want to store an example to play some unitests and I don't want it to be reformated

Another one, but quite more specific... for an anonymous customer in v16, they have terrible printers and the native barcode generator from Odoo does not work well for them, I have a big javascript copy pasted that generates barcode in a module.
=> Maybe it could be reformated with no issue but I prefer not even touch it...

Also, in the current exclude, there is stuff from local-src (.html, cassettes, README, etc) so I guess we'll still want to exclude all this if we include only local-src. (I just want to be sure both are compatible (exclude and include)

[florian-dacosta]

you need also to update this file: update_oca_file.py

Not sure how/what I should update

[hparfr]

These pre-commit files are derived from oca, so instead of patching the result of the script, you can patch in update_oca_file

your example makes sense. I guess, we want to put the exclusion closer to its place (ie: next to local-src/some_modules/data/copypastedfile )