Add support for Path arguments to more methods

[straight-shoota]

Closes #7688

[Sija]

How about Dir.current returning Path?

[straight-shoota]

@Sija Yeah that should be the goal. But it's also a breaking change. I want to ship the non-breaking changes to prepare the API first.
There are also a lot of other methods on Dir and File that should return Path, not just Dir.current.

[straight-shoota]

Rebased on master and squashed. Needs an approval again.

[RX14]

Can we get rid of the force push dismiss? It's really useful to know whether past reviews were 👍 or 👎 before the force push, and this setting completely destroys that info.

[waj]

Can we get rid of the force push dismiss? It's really useful to know whether past reviews were 👍 or 👎 before the force push, and this setting completely destroys that info.

@RX14 the setting says "Dismiss stale pull request approvals when new commits are pushed". So isn't just 👍 that are dismissed?

[RX14]

It seems to do both, and everyone seems to dislike the setting in practice.

With better UX and better semantics from github, it could be a wonderful feature, but in practice, right now, it seems to do more harm than good.

[waj]

I don’t get it. It’s just approvals that get dismissed but comments are still there.

[waj]

Oh you mean if someone request changes you don’t see the red flag anymore after the push?

[asterite]

I think he means it's annoying having to reapprove things when minor fixes are. made.

[straight-shoota]

We should better discuss this somewhere else. Maybe on the forum.

[RX14]

@asterite she means both what you and waj said :)