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

[Versions-*.plist.in] Define Feature keys #1880

Merged
merged 1 commit into from Mar 17, 2017
Merged

Conversation

jstedfast
Copy link
Member

No description provided.

@jstedfast jstedfast added this to the d15-1 milestone Mar 16, 2017
@monojenkins
Copy link
Collaborator

Build success

@spouliot spouliot added the requires-qa-before-merge The pull request requires QA to approve it before it can be merged label Mar 16, 2017
@spouliot
Copy link
Contributor

@rolfbjarne
Copy link
Member

rolfbjarne commented Mar 17, 2017

XS has already been updated to use these feature keys, which means that the corresponding features won't work until this has been merged (https://bugzilla.xamarin.com/show_bug.cgi?id=52717#c19)

<string>mlaunch-install-progress</string>
<string>http-client-handlers</string>
<string>mono-symbol-archive</string>
<string>sgen-concurrent-gc-experimental</string>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's tracking one-too-many issues. @jstedfast can you remove both <string>sgen-concurrent-gc-experimental</string> lines from the PR so the other feature can be tested. We'll do another one once for the last item once we have results from QA. Thanks.

Copy link
Contributor

@spouliot spouliot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@spouliot spouliot merged commit 0e9e791 into d15-1 Mar 17, 2017
@spouliot spouliot deleted the d15-1-sgen-concurrent-gc branch March 17, 2017 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requires-qa-before-merge The pull request requires QA to approve it before it can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants