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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow/Enable configurations for ng deploy commands #17332

Closed
1 of 16 tasks
george43g opened this issue Mar 29, 2020 · 3 comments 路 Fixed by #17336
Closed
1 of 16 tasks

Allow/Enable configurations for ng deploy commands #17332

george43g opened this issue Mar 29, 2020 · 3 comments 路 Fixed by #17336
Assignees
Labels
area: angular/cli freq1: low Only reported by a handful of users who observe it rarely severity3: broken type: bug/fix
Milestone

Comments

@george43g
Copy link

馃悶 Bug report

Command (mark with an x)

  • new
  • build
  • serve
  • deploy
  • test
  • e2e
  • generate
  • add
  • update
  • lint
  • xi18n
  • run
  • config
  • help
  • version
  • doc

Is this a regression?

Probably not

Description

Not sure if this is a bug or a feature request.

Allow specifying a --configuration='production' for ng deploy commands.
This should simply read from the corresponding configuration and apply those settings on top of settings found in options under the deploy config.

According to docs, this should work for other types of tasks such as build/serve/lint: https://angular.io/guide/cli-builder#target-configuration

馃敩 Minimal Reproduction

"deploy": {
          "builder": "@angular/fire:deploy",
          "options": {
            "buildTarget": "my-project:build",
            "firebaseProject": "dev"
          },
          "configurations": {
            "production": {
              "buildTarget": "my-project:build:production",
              "firebaseProject": "prod"
            }
          }
        }

This command works:
ng deploy myProject

This command does not work, even with correct configuration:
ng deploy myProject --configuration production

馃敟 Exception or Error

Unknown option: '--configuration'
Unknown option: 'production'

馃實 Your Environment

Angular CLI 9

@alan-agius4 alan-agius4 added area: angular/cli freq1: low Only reported by a handful of users who observe it rarely type: bug/fix labels Mar 29, 2020
@ngbot ngbot bot added this to the needsTriage milestone Mar 29, 2020
@ngbot ngbot bot modified the milestones: needsTriage, Backlog Mar 29, 2020
@alan-agius4 alan-agius4 self-assigned this Mar 29, 2020
@alan-agius4
Copy link
Collaborator

@george43g, thanks for reporting this.

Indeed this is a bug which should be trivial to fix. I鈥檒l take a look at it tomorrow!

@george43g
Copy link
Author

Awesome! I'll let some package maintainers know about this. Should result in some cleaner code/setups. Hopefully, the pull request will be merged in soon.

kyliau pushed a commit that referenced this issue Mar 30, 2020
Previously we only allowed `--project` and `--help` as a valid options. With this change we also allow `--configuration`

Fixes #17332
kyliau pushed a commit that referenced this issue Mar 30, 2020
Previously we only allowed `--project` and `--help` as a valid options. With this change we also allow `--configuration`

Fixes #17332
@angular-automatic-lock-bot
Copy link

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-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Apr 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area: angular/cli freq1: low Only reported by a handful of users who observe it rarely severity3: broken type: bug/fix
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants