bakery icon indicating copy to clipboard operation
bakery copied to clipboard

Documentation missing when out param

Open bjorkstromm opened this issue 8 years ago • 6 comments

related to #42 which fixed code gen. Documentation is still however missing.

Reproduce with this alias.

public System.Int32 StartProcess(Cake.Core.IO.FilePath fileName, Cake.Core.IO.ProcessSettings settings, out System.Collections.Generic.IEnumerable<System.String> redirectedOutput)
{
    return Cake.Common.ProcessAliases.StartProcess(Context, fileName, settings, out redirectedOutput);
}

ping @patriksvensson

bjorkstromm avatar Jul 30 '17 21:07 bjorkstromm

@mholo65 We probably don't generate the CRef for the method signature correctly which makes us not find the docs in the XML file.

patriksvensson avatar Jul 30 '17 22:07 patriksvensson

@patriksvensson yep. Do you want to fix?

bjorkstromm avatar Jul 30 '17 22:07 bjorkstromm

Is this on me? If so, feel free to assign. Hoping to carve out some hours soon to clear my backlog of Cake docs issues.

daveaglick avatar Jul 30 '17 22:07 daveaglick

@daveaglick Lucky for you, this isn't the website but the bakery app 😉

patriksvensson avatar Jul 30 '17 22:07 patriksvensson

but however, @daveaglick feel free to send PR if you like :wink:

bjorkstromm avatar Jul 30 '17 22:07 bjorkstromm

Lol, well then, carry on!

daveaglick avatar Jul 30 '17 22:07 daveaglick