Avoid unnecessary heap allocation and copy

[ryco117]

Looks like the current implementation of From<FileHandle> for PathBuf passes a Path reference to the PathBuf::from which results in a new heap allocation from a call to to_os_string() down the call-stack.

Since From<FileHandle> consumes the FileHandle it seems reasonable to consume the member PathBuf and return it to the caller without any copies or allocations.

[ryco117]

The error says that the changelog check can be avoided by adding the no changelog label. This seems reasonable since the change will not be visible.

[PolyMeilex]

Thanks!