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

Create and update example project on launch #315

Merged
merged 4 commits into from
Oct 30, 2015
Merged

Create and update example project on launch #315

merged 4 commits into from
Oct 30, 2015

Conversation

2fast2fourier
Copy link
Contributor

What's this PR do?

This PR adds an examples folder with source files on launch. This will automatically add new files if they exist in the examples directory but are not in the user's example project, allowing new files to be added later.

How should this be tested by the reviewer?

Launch the app and verify that the Examples project exists and is populated. Add a new example and rebuild, verify that it is added after the fact.

What are the relevant tickets?

Closes #295

@@ -70,6 +83,7 @@ function onRender(err){
} = app;

const {
ensureExampleProject,
Copy link
Contributor

Choose a reason for hiding this comment

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

no longer used

@phated
Copy link
Contributor

phated commented Oct 30, 2015

@PropGit a warning on this feature... it is adding quite a bit of time to the startup of the application when the examples directory doesn't exist because it is creating all those files before first render (which makes sense).

phated added a commit that referenced this pull request Oct 30, 2015
Create and update example project on launch
@phated phated merged commit b730886 into master Oct 30, 2015
@phated phated deleted the examples-dir branch October 30, 2015 03:23
@PropGit
Copy link
Contributor

PropGit commented Oct 30, 2015

@phated - noted, thanks. We'll perhaps add a prompt message ahead of that scenario on a future release.

@phated phated removed their assignment Jan 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants