Fixes #57: Improves inheritdoc so that developer documentation is properly referenced on the autogenerated interfaces

[ChaseFlorell]

based on the comments in #57, by using a cref back to the original implementation, we are able to get intellisense working off of autogenerated interfaces.

Resolves #57

[simonmckenzie]

Hi @ChaseFlorell,

There seem to be a lot of whitespace changes - have you run csharpier? Usually the pipeline runs for pull requests and I think you'd have received a lint failure notification - I don't know why it didn't today.

[ChaseFlorell]

Hi @ChaseFlorell,

There seem to be a lot of whitespace changes - have you run csharpier? Usually the pipeline runs for pull requests and I think you'd have received a lint failure notification - I don't know why it didn't today.

I have not, I'm not sure how to run it TBH. I'll take a look

[simonmckenzie]

Hi @ChaseFlorell,
There seem to be a lot of whitespace changes - have you run csharpier? Usually the pipeline runs for pull requests and I think you'd have received a lint failure notification - I don't know why it didn't today.

I have not, I'm not sure how to run it TBH. I'll take a look

This is what's in the pipeline - you should be able to run the same locally.

dotnet tool restore
dotnet csharpier --check .

... I think running without --check will format the files in place.

[ChaseFlorell]

Hi @ChaseFlorell,
There seem to be a lot of whitespace changes - have you run csharpier? Usually the pipeline runs for pull requests and I think you'd have received a lint failure notification - I don't know why it didn't today.

I have not, I'm not sure how to run it TBH. I'll take a look

This is what's in the pipeline - you should be able to run the same locally.

dotnet tool restore
dotnet csharpier --check .

... I think running without --check will format the files in place.

Got it, thanks

[ChristianSauer]

For first time contributors, package maintainers (only me currently) have to approve the pipeline run.
Thats due to security concerns (imagine replacing the pipeline with a bitcoin mining script...)

I really like the PR, if you can get it "green" I will merge it asap

[ChaseFlorell]

For first time contributors, package maintainers (only me currently) have to approve the pipeline run. Thats due to security concerns (imagine replacing the pipeline with a bitcoin mining script...)

I really like the PR, if you can get it "green" I will merge it asap

I'm not quite sure what's missing. I ran > dotnet-csharpier . over the solution on my last commit.

[ChaseFlorell]

I'm confused because CI reports the following

Whereas my local installation of chsharpier shows

[ChaseFlorell]

For first time contributors, package maintainers (only me currently) have to approve the pipeline run. Thats due to security concerns (imagine replacing the pipeline with a bitcoin mining script...)

I really like the PR, if you can get it "green" I will merge it asap

@ChristianSauer I did push a new version, I'm wondering if you can rerun the pipeline?

[ChristianSauer]

I finally had time to merge this. Thanks for your patience!