Hi folks, In this weeks "PyCoders" newsletter, they note that "git blame" can now ignore revisions. https://twitter.com/llanga/status/1163767833706323970 https://git-scm.com/docs/git-blame#Documentation/git-blame.txt---ignore-revs... I believe this might remove one of the biggest objections we've had to things like running a code formatter such as black. It's not obvious to me that there won't be additional issues introduced by doing this, but it could be a good starting place for consistent code formatting. -Matt
Hi Matt,
I think it's worth giving this a shot. Does anyone know of any bots or
other automated ways of applying this to pull requests?
Britton
On Tue, Aug 20, 2019 at 8:37 PM Matthew Turk
Hi folks,
In this weeks "PyCoders" newsletter, they note that "git blame" can now ignore revisions.
https://twitter.com/llanga/status/1163767833706323970
https://git-scm.com/docs/git-blame#Documentation/git-blame.txt---ignore-revs...
I believe this might remove one of the biggest objections we've had to things like running a code formatter such as black. It's not obvious to me that there won't be additional issues introduced by doing this, but it could be a good starting place for consistent code formatting.
-Matt _______________________________________________ yt-dev mailing list -- yt-dev@python.org To unsubscribe send an email to yt-dev-leave@python.org
For unyt I just have a test in the test suite that checks to make sure if
black has output and if it does, it fails the test:
https://github.com/yt-project/unyt/blob/41bf39cbc73e85468d2aea780b5a88ae218e...
There is also this:
https://github.com/Mariatta/black_out
On Tue, Aug 27, 2019 at 4:16 AM Britton Smith
Hi Matt,
I think it's worth giving this a shot. Does anyone know of any bots or other automated ways of applying this to pull requests?
Britton
On Tue, Aug 20, 2019 at 8:37 PM Matthew Turk
wrote: Hi folks,
In this weeks "PyCoders" newsletter, they note that "git blame" can now ignore revisions.
https://twitter.com/llanga/status/1163767833706323970
https://git-scm.com/docs/git-blame#Documentation/git-blame.txt---ignore-revs...
I believe this might remove one of the biggest objections we've had to things like running a code formatter such as black. It's not obvious to me that there won't be additional issues introduced by doing this, but it could be a good starting place for consistent code formatting.
-Matt _______________________________________________ yt-dev mailing list -- yt-dev@python.org To unsubscribe send an email to yt-dev-leave@python.org
_______________________________________________ yt-dev mailing list -- yt-dev@python.org To unsubscribe send an email to yt-dev-leave@python.org
Hi Britton, I've seen a lot of projects using "pre-commit" [1] You basically pip install it, add a config yaml file [2] and forget about the whole thing ;-) Cheers, Kacper [1] https://pre-commit.com/ [2] https://raw.githubusercontent.com/jupyter/repo2docker/master/.pre-commit-con... On 8/27/19 4:16 AM, Britton Smith wrote:
Hi Matt,
I think it's worth giving this a shot. Does anyone know of any bots or other automated ways of applying this to pull requests?
Britton
On Tue, Aug 20, 2019 at 8:37 PM Matthew Turk
mailto:matthewturk@gmail.com> wrote: Hi folks,
In this weeks "PyCoders" newsletter, they note that "git blame" can now ignore revisions.
https://twitter.com/llanga/status/1163767833706323970 https://git-scm.com/docs/git-blame#Documentation/git-blame.txt---ignore-revs...
I believe this might remove one of the biggest objections we've had to things like running a code formatter such as black. It's not obvious to me that there won't be additional issues introduced by doing this, but it could be a good starting place for consistent code formatting.
-Matt _______________________________________________ yt-dev mailing list -- yt-dev@python.org mailto:yt-dev@python.org To unsubscribe send an email to yt-dev-leave@python.org mailto:yt-dev-leave@python.org
_______________________________________________ yt-dev mailing list -- yt-dev@python.org To unsubscribe send an email to yt-dev-leave@python.org
Hi folks,
OK, so there's sorta of medium support for it! I've been
experimenting with the various things and I *think* this could work.
Still, we'd need to basically do a flag day on the code, *or* go
through and manually reformat all the code out there in extant pull
requests.
Should we discuss at an upcoming team meeting, but with the tentative
plan to evaluate it as a go?
On Tue, Aug 27, 2019 at 8:36 AM Kacper Kowalik
Hi Britton, I've seen a lot of projects using "pre-commit" [1] You basically pip install it, add a config yaml file [2] and forget about the whole thing ;-) Cheers, Kacper
[1] https://pre-commit.com/ [2] https://raw.githubusercontent.com/jupyter/repo2docker/master/.pre-commit-con...
On 8/27/19 4:16 AM, Britton Smith wrote:
Hi Matt,
I think it's worth giving this a shot. Does anyone know of any bots or other automated ways of applying this to pull requests?
Britton
On Tue, Aug 20, 2019 at 8:37 PM Matthew Turk
mailto:matthewturk@gmail.com> wrote: Hi folks,
In this weeks "PyCoders" newsletter, they note that "git blame" can now ignore revisions.
https://twitter.com/llanga/status/1163767833706323970 https://git-scm.com/docs/git-blame#Documentation/git-blame.txt---ignore-revs...
I believe this might remove one of the biggest objections we've had to things like running a code formatter such as black. It's not obvious to me that there won't be additional issues introduced by doing this, but it could be a good starting place for consistent code formatting.
-Matt _______________________________________________ yt-dev mailing list -- yt-dev@python.org mailto:yt-dev@python.org To unsubscribe send an email to yt-dev-leave@python.org mailto:yt-dev-leave@python.org
_______________________________________________ yt-dev mailing list -- yt-dev@python.org To unsubscribe send an email to yt-dev-leave@python.org
_______________________________________________ yt-dev mailing list -- yt-dev@python.org To unsubscribe send an email to yt-dev-leave@python.org
participants (4)
-
Britton Smith
-
Kacper Kowalik
-
Matthew Turk
-
Nathan