This issue concerns devs on this repo.
I believe the dist/ folder and the docs/docs/dist folders should be added to .gitignore and any grunt command to build or clean MUST not pollute the diff with unrelated changes.
This should help make PRs easier to work with because of smaller diffs, and it's also good practice to do so.
If this change is accepted, you might need to change your npm publish process and add a build step (or not).
@caseyjhol Is there a good reason to keep the built files in the repo?
If not, I can make a PR.
This issue concerns devs on this repo.
I believe the
dist/folder and thedocs/docs/distfolders should be added to.gitignoreand any grunt command to build or clean MUST not pollute the diff with unrelated changes.This should help make PRs easier to work with because of smaller diffs, and it's also good practice to do so.
If this change is accepted, you might need to change your npm publish process and add a build step (or not).
@caseyjhol Is there a good reason to keep the built files in the repo?
If not, I can make a PR.