Skip to content

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

Merged
spouliot merged 1 commit into
d15-1from
d15-1-sgen-concurrent-gc
Mar 17, 2017
Merged

[Versions-*.plist.in] Define Feature keys#1880
spouliot merged 1 commit into
d15-1from
d15-1-sgen-concurrent-gc

Conversation

@jstedfast

Copy link
Copy Markdown
Member

No description provided.

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

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor

@rolfbjarne

rolfbjarne commented Mar 17, 2017

Copy link
Copy Markdown
Member

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)

Comment thread Versions-ios.plist.in
<string>mlaunch-install-progress</string>
<string>http-client-handlers</string>
<string>mono-symbol-archive</string>
<string>sgen-concurrent-gc-experimental</string>

Copy link
Copy Markdown
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.

@spouliot spouliot left a comment

Copy link
Copy Markdown
Contributor

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.

6 participants