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

feat(compildconfig): support multiple projects #229

Merged
merged 4 commits into from
Jan 24, 2020

Conversation

SanderElias
Copy link
Contributor

@SanderElias SanderElias commented Jan 23, 2020

This Pr adds the --project option to Scully. This makes it possible to support multiple projects
in 1 cli app

BREAKING CHANGE: The scully.config.js is now named scully.projectName.config.js You need to
manually update the name.

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Other... Please describe:

What is the current behavior?

We only support the default project in a CLI app

Issue Number: N/A

What is the new behavior?

by using the --project cmd line option you can now use all projects in the app

Does this PR introduce a breaking change?

  • Yes
  • No

rename the scully.config.js to scully.<projects name>.config.js where <project name> is the name your project has in the angular.json file.

SanderElias and others added 4 commits January 23, 2020 17:53
This Pr adds the `--project` option to Scully. This makes it possible to support multiple projects
in 1 cli app

BREAKING CHANGE: The `scully.config.js` is now named `scully.projectName.config.js` You need to
manually update the name.
@SanderElias
Copy link
Contributor Author

@Villanuevand I added some docs in here, but they still need work.

@SanderElias SanderElias merged commit bf046c7 into master Jan 24, 2020
@SanderElias SanderElias deleted the Sander/HandleMultipleProjectsInSully branch January 24, 2020 11:57
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.

2 participants