chemeris/UHD-Fairwaves — comment on commit 92dd9d4a44680625837edcef17a6af0845d9e09d
File:  line 0
By smunaut:

A few comments:

- This patch obviously includes several unrelated changes all bundled into one which isn't good practice. In this case the clk changes and git ignore should be separate.
- What's up with the renames of .ngc ?
- You seem to have included backup files by mistage (the coregen~ file)

If you don't know it, you should look into git citool which allows to graphically pick and review changes to put in a commit.
Using git commit -v (verbose) also allows to get a final view of the patch before commit.