Skip to content

Implemented Uwp ScriptStore and PreparedScriptStore to allow caching scripts as bytecdoes - #2653

Merged
14 commits merged into
microsoft:masterfrom
jnvbui:users/ngbu/implement_chakra_store
Jun 21, 2019
Merged

Implemented Uwp ScriptStore and PreparedScriptStore to allow caching scripts as bytecdoes#2653
14 commits merged into
microsoft:masterfrom
jnvbui:users/ngbu/implement_chakra_store

Conversation

@jnvbui

@jnvbui Joseph Bui (jnvbui) commented Jun 20, 2019

Copy link
Copy Markdown
Contributor

Set settings.EnableByteCodeCacheing to true in the ReactInit.cpp file to enable this feature. Default will be false.

Helps with reducing boot up time.

Microsoft Reviewers: Open in CodeFlow

@jnvbui
Joseph Bui (jnvbui) requested a review from a team as a code owner June 20, 2019 17:46
@ghost ghost added the vnext label Jun 20, 2019
@msftclas

Microsoft Contribution License Agreements (msftclas) commented Jun 20, 2019

Copy link
Copy Markdown

CLA assistant check
All CLA requirements met.

@stecrain

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines successfully started running 1 pipeline(s).

@stecrain

Copy link
Copy Markdown
Contributor

I like this PR but I also worked with Joseph on it. So another set of eyes would be great

Comment thread vnext/ReactUWP/UwpPreparedScriptStore.cpp Outdated
Comment thread vnext/include/ReactUWP/IReactInstance.h
Comment thread vnext/ReactUWP/UwpPreparedScriptStore.h

@mganandraj Anandraj (mganandraj) 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.

:shipit:

@jnvbui

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Commenter does not have sufficient privileges for PR 2653 in repo microsoft/react-native-windows

@stecrain

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines successfully started running 1 pipeline(s).

@stecrain

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines successfully started running 1 pipeline(s).

@stecrain Stephen Crain (stecrain) added the AutoMerge Causes a PR to be automatically merged once all requirements are passed (label drives bot activity) label Jun 20, 2019
@ghost

Copy link
Copy Markdown

Hello Stephen Crain (@stecrain)!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AutoMerge Causes a PR to be automatically merged once all requirements are passed (label drives bot activity)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants