Skip to content
This repository has been archived by the owner on Nov 26, 2024. It is now read-only.

"Overload:" artifact in "Assertions" article #106

Closed
vasily-kirichenko opened this issue May 26, 2016 · 6 comments
Closed

"Overload:" artifact in "Assertions" article #106

vasily-kirichenko opened this issue May 26, 2016 · 6 comments
Assignees
@dend
Labels
bug
Milestone
JUN-2016

Comments

@vasily-kirichenko
Copy link
Contributor

https://msdn.microsoft.com/en-us/visualfsharpdocs/conceptual/assertions-[fsharp]

image

I'm not sure, but it seems that Overload:System.Diagnostics.Debug.Assert should be a link.
@dend dend added the bug label May 26, 2016
@dend dend added this to the JUN-2016 milestone May 26, 2016
@dend dend self-assigned this May 26, 2016
@ReedCopsey
Copy link
Contributor

I'll add "Overload:" to the list of bad prefixes I remove in #61

@ReedCopsey ReedCopsey mentioned this issue May 26, 2016
Closed
@ReedCopsey
Copy link
Contributor

@dend or @vasily-kirichenko Please close this now. Fixed.

@dend
Copy link
Contributor

dend commented May 26, 2016

@ReedCopsey there looks to be a note formatting issue. I will fix that and will update the bug after.

@ReedCopsey
Copy link
Contributor

@dend So - are the braces actually required?

@dend
Copy link
Contributor

dend commented May 26, 2016

@ReedCopsey nope, it's the location of the exclamation mark and the prefix.

@ReedCopsey
Copy link
Contributor

@dend Does it need a space between the > and ! ? I'll submit a PR to fix, if that's the issue.

@dend dend closed this as completed in 7fce179 May 29, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@dend dend

Labels
bug
Projects
None yet
Milestone
JUN-2016
Development

No branches or pull requests
3 participants
@vasily-kirichenko @dend @ReedCopsey