-
Notifications
You must be signed in to change notification settings - Fork 559
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
Update RC1 release notes #570
Conversation
hongdai
commented
Dec 2, 2015
- The scenario table are updated according to test results on Asp.Net 5 and UWP.
If you have issue numbers, would you mind adding a "more..." hyperlink to it like the other places with known issues? |
I will add the issue numbers I am aware of. |
@@ -28,9 +28,9 @@ Supported features | |||
||NetHttpBinding|:white_check_mark: | :white_check_mark: | :white_check_mark: | :warning: [more...](https://github.com/dotnet/wcf/issues/534) | | |||
||NetTcpBinding|:white_check_mark: | :white_check_mark: | :white_check_mark: | :warning: [more...](https://github.com/dotnet/wcf/issues/534) | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can "more..." point to multiple issues (if there are more than one)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could make "more..." a query, but I couldn't find any clean way to form accurate queries unique to the feature for that row in the table. I think it might be reasonable to make the NET Native and Linux columns use a query for labels "bug" and "NET Native" (or "Linux"). What do you think of adding new labels that correspond to the cells in the "Feature" column? It would allow self-maintaining queries and give us a better way to indicate "what feature is affected by this bug?".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will need a query for RC1 only bugs. The issues status reflects the latest status,
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point. Let's put more thoughts in this and not block this PR by it.
BTW, can we update the OS X column from
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, let's make OSX all ❓. And I agree, let's merge the PR and continue refining this table as we decide on improvements.
Issue #567 recommends changing the legend, which would affect many of the cells in this table. I like the proposed legend changes and recommend we expand this PR to that if fixes the legend as well as all the cells. We can make a separate PR to convert all the "more..." links to queries. |
:x: -- Not supported | ||
:white_check_mark: -- Works with no known major issues | ||
:warning: -- Partially works with known issues or only partially tested | ||
:x: -- Does not work | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NIT: remove empty line
df45087
to
b61253c
Compare
|MessageVersion|SOAP 1.1 UTF8|:white_check_mark: | :white_check_mark: | :white_check_mark: | :question: | | ||
||SOAP 1.2 UTF8|:white_check_mark: | :white_check_mark: | :white_check_mark: | :question: | | ||
|Contracts | DataContract | :warning:[more...](https://github.com/dotnet/wcf/issues/325) | :white_check_mark: | :white_check_mark: | :question: | | ||
| | XmlSerializer | :white_check_mark: | :white_check_mark: | :white_check_mark: | :question: | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sense. I'll update it.
* The scenario table are updated according to test results on Asp.Net 5 and UWP.
LGTM. Thanks @hongdai. |
LGTM |