skpkg: fix codespell errors#133
skpkg: fix codespell errors#133kwagoner94 wants to merge 0 commit intodiffpy:migrationfrom kwagoner94:pre-commit-spelling
Conversation
sbillinge
left a comment
There was a problem hiding this comment.
This seems to be a mix of black and codespell changes. I wonder if something went wrong with the workflow. I don't want to create extra work, but it would sure be easier to review if they were on separate PRs. Is it possible that you ran black but not on all files and directories? SOmething like htat.
Maybe create a new branch off a synced main and try and run black on everything and see what happens. If we can regenerate the white-space and single-double quote fixes, I can merge them, then we can merge that into this branch.
It may be better if you could pick a tier-2 project as a first one where any mistakes we make are less consequential?
|
I created a new branch off a synced main and ran black on everything. All files were left unchanged. Should I just re do the |
|
Can we switch to another project and then come back to this one? The only files that should be modified on this one should be the handful that had typo's caught by codespell, not 102 files. Were you very selective about the files that you added to the commit? Kind of one at a time as you worked on them? |
|
I'm not sure what I did wrong in my workflow. I can go back and start over tomorow by editing the files with the codespell errors. |
|
there's probably a way of reocoving the work you did so it is not lost (it will be on a different clean branch) but as a first cut starting over but working on a tier-2 project as a first one would be my preference. There is a learning curve, but this goes very quickly once over the hump. |
|
Do you have a tier 2 project in mind that I could work on? |
|
@kwagoner94 how about trying diffpy.srmise? |
Fixed codespell erros and suppressed known terms (e.g., DISCUS, CONECT, Nd, Te, BU, discus, etc.) using --ignore-regex in .pre-commit-config.yaml.