Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documentation missing when out param #48

Open
bjorkstromm opened this issue Jul 30, 2017 · 6 comments
Open

Documentation missing when out param #48

bjorkstromm opened this issue Jul 30, 2017 · 6 comments
Labels

Comments

@bjorkstromm
Copy link
Member

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

@patriksvensson
Copy link
Member

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

@bjorkstromm
Copy link
Member Author

@patriksvensson yep. Do you want to fix?

@daveaglick
Copy link
Member

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.

@patriksvensson
Copy link
Member

patriksvensson commented Jul 30, 2017

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

@bjorkstromm
Copy link
Member Author

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

@daveaglick
Copy link
Member

Lol, well then, carry on!

@bjorkstromm bjorkstromm added this to the 0.2.0 milestone Sep 19, 2017
@bjorkstromm bjorkstromm removed this from the 0.2.0 milestone Dec 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants