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

Fix c# extension methods. #339

Merged
merged 10 commits into from
Sep 28, 2015
Merged

Fix c# extension methods. #339

merged 10 commits into from
Sep 28, 2015

Conversation

matthid
Copy link
Member

@matthid matthid commented Sep 20, 2015

This fixes #201 by updating Yaaf.FSharp.Scripting and makes [omit] work, even when markDownComments is set to false.
I also enabled AppVeyor and added a appveyor.yml file, will test it as part of this PR.

…annot get the attributes of the current member.
… Fixes crash with C# extension methods when the assembly containing them was build against net45. Fixes #201. (Might fix) References #328, #325, #139
@matthid
Copy link
Member Author

matthid commented Sep 20, 2015

@tpetricek OK I think the owner of the repro needs to add the webhook (http://help.appveyor.com/discussions/questions/203-auto-run-tests-on-pull-requests#comment_33212404). You should be able to login into AppVeyor with your github account and then select "matthid" then you should see the project relevant settings (the webhook url). I hope I got your E-Mail address right in AppVeyor.

Alternatively you can login into AppVeyor with your Github account, select your own account and then click the "add project" button on FSharp.Formatting. I think in this scenario AppVeyor will automatically add the hook to GitHub.

@tpetricek
Copy link
Member

All changes look good to me - feel free to merge this & release a new non-alpha!

matthid added a commit that referenced this pull request Sep 28, 2015
@matthid matthid merged commit 9290266 into master Sep 28, 2015
@matthid matthid deleted the fix_extension_methods branch December 17, 2015 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reference documentation generation with CSharp Extension methods fails
2 participants