Skip to content

feat(config): ngConfig - #452

Closed
jkuri wants to merge 1 commit into
angular:masterfrom
jkuri:ng-config
Closed

feat(config): ngConfig#452
jkuri wants to merge 1 commit into
angular:masterfrom
jkuri:ng-config

Conversation

@jkuri

Copy link
Copy Markdown
Contributor

No description provided.

@jkurijkuri added feature Label used to distinguish feature request from other issues pr_action: review labels Apr 18, 2016
Comment threadaddon/ng2/commands/set.ts Outdated
config.set(rawArgs[0], rawArgs[1], commandOptions.force);
let config = new Config();
config.validatePath(rawArgs[0]);
config.set(config.config, rawArgs[0], rawArgs[1]);

@hanslhanslApr 18, 2016

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.

Could you do a a=b kinda of syntax?

Basically:

for(letargofrawArgs){let[key,value]=arg.split('=',1);config.validatePath(key);config.set(key,value);}

What do you think? This way we can ng config set key[1]=value1 key[2]=value2.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Good point, thanks. Done.

@hansl

Copy link
Copy Markdown
Contributor

Linting and some tests are failing.

@jkurijkuri closed this Apr 21, 2016
@jkuri
jkuri deleted the ng-config branch April 21, 2016 17:14
@Brocco

Copy link
Copy Markdown
Contributor

@jkuri You'll also want to pull in the changes I have in this PR (#468) as part of this PR. It will expose the configuration on the project object for access within our commands and blueprints.

@jkuri

Copy link
Copy Markdown
ContributorAuthor

@Brocco thanks, I would missed that.

clydin pushed a commit that referenced this pull request Jun 5, 2018
it has a transitive dep on an updated tsickle that picks up a fix in source-map typings
hansl pushed a commit that referenced this pull request Jun 6, 2018
it has a transitive dep on an updated tsickle that picks up a fix in source-map typings
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-botangular-automatic-lock-botBot locked and limited conversation to collaborators Sep 9, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

featureLabel used to distinguish feature request from other issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@jkuri@hansl@Brocco