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

Misc grooming and updates #474

Merged
merged 7 commits into from
May 27, 2019
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 1 addition & 4 deletions MAINTAINERS.md
Original file line number Diff line number Diff line change
@@ -38,10 +38,7 @@ Then follow these steps:
7. Run `npm run changelog`. This will create a commit with the new updated `CHANGELOG.md`, but it will not be pushed so
you can check it up before pushing.

8. Rebase `ci/*` branches onto `master` (could be automated). This should also be done regularly outside release cycle,
after merging new PR for example.

9. Done! :tropical_drink: Now you can tell the world a new version is out! :speaker:
8. Done! :tropical_drink: Now you can tell the world a new version is out! :speaker:

## Dedicated CI branches

2 changes: 0 additions & 2 deletions cli/index.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
'use strict';

const path = require('path');
const fs = require('fs');
const child = require('child_process');
2 changes: 0 additions & 2 deletions generators/addon/index.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
'use strict';

const chalk = require('chalk');
const Insight = require('insight');
const camelCase = require('lodash.camelcase');
2 changes: 0 additions & 2 deletions generators/addon/templates/generators/app/_index.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
'use strict';

const Generator = require('@ngx-rocket/core');
<% if (props.advanced) { -%>
const chalk = require('chalk');
15 changes: 7 additions & 8 deletions generators/app/index.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
'use strict';

const fs = require('fs');
const path = require('path');
const chalk = require('chalk');
@@ -43,9 +41,10 @@ class NgxGenerator extends Generator {
this.props.location = this.options['location-strategy'];
}

if (this.options.strict) {
this.props.strict = true;
}
this.props.strict = this.options.strict;
this.props.skipInstall = this.options['skip-install'];
this.props.skipQuickstart = this.options['skip-quickstart'];
this.props.initGit = this.options.git;

// Updating
let fromVersion = null;
@@ -115,11 +114,11 @@ class NgxGenerator extends Generator {
}

install() {
if (this.options.git) {
if (this.props.initGit) {
this.spawnCommandSync('git', ['init', '--quiet']);
}

if (!this.options['skip-install']) {
if (!this.props.skipInstall) {
this.log(`\nRunning ${chalk.yellow(`${this.packageManager} install`)}, please wait...`);

const install = this.packageManager === 'yarn' ? this.yarnInstall.bind(this) : this.npmInstall.bind(this);
@@ -146,7 +145,7 @@ class NgxGenerator extends Generator {
return;
}

if (this.options['skip-quickstart']) {
if (this.props.skipQuickstart) {
return;
}

2 changes: 0 additions & 2 deletions generators/app/options.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
'use strict';

module.exports = [
{
name: 'skip-welcome',
2 changes: 0 additions & 2 deletions generators/app/prompts.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
'use strict';

module.exports = [
{
type: 'input',
2 changes: 1 addition & 1 deletion generators/app/templates/_package.json
Original file line number Diff line number Diff line change
@@ -113,7 +113,7 @@
<% } -%>
"core-js": "^2.6.3",
"rxjs": "^6.4.0",
"zone.js": "^0.9.0"
"zone.js": "~0.8.26"
},
"devDependencies": {
<% if (props.target.includes('cordova')) { -%>
2 changes: 0 additions & 2 deletions generators/app/templates/proxy.conf.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
'use strict';

const HttpsProxyAgent = require('https-proxy-agent');

/*
Original file line number Diff line number Diff line change
@@ -18,9 +18,10 @@ export class ShellComponent implements OnInit, OnDestroy {

ngOnInit() {
// Automatically close side menu on screens > sm breakpoint
this.media.media$
this.media
.asObservable()
.pipe(
filter((change: MediaChange) => (change.mqAlias !== 'xs' && change.mqAlias !== 'sm')),
filter((changes: MediaChange[]) => changes.some(change => change.mqAlias !== 'xs' && change.mqAlias !== 'sm')),
untilDestroyed(this)
)
.subscribe(() => this.sidenav.close());
13 changes: 0 additions & 13 deletions generators/app/templates/src/app/shell/shell.module.spec.ts

This file was deleted.

12 changes: 6 additions & 6 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -63,7 +63,7 @@
"lodash.get": "^4.4.2",
"lodash.upperfirst": "^4.3.1",
"minimist": "^1.2.0",
"semver": "^6.0.0",
"semver": "^6.1.0",
"update-notifier": "^3.0.0",
"yeoman-environment": "^2.3.4"
},