Skip to content

Conversation

@bobbai00
Copy link
Contributor

@bobbai00 bobbai00 commented Sep 20, 2024

This PR updates the ng build command in deploy-daemon.sh.

Before the change, the command is:
ng build --prod --deploy-url=/ --base-href=/

However, --prod is deprecated in angular 14 according to https://github.com/angular/angular-cli/blob/main/CHANGELOG.md#breaking-changes-2
Screenshot 2024-09-19 at 10 02 56 PM

And our angular version is angular 16.

So I updated it to make it align with current angular: ng build --configuration production --deploy-url=/ --base-href=/

This PR updates the `ng build` command in `deploy-daemon.sh`.

Before the change, the command is:
`ng build --prod  --deploy-url=/ --base-href=/`

However, `--prod` is deprecated according to https://github.com/angular/angular-cli/blob/main/CHANGELOG.md#breaking-changes-2

So I updated it to make it align with current angular:
`ng build --configuration production  --deploy-url=/ --base-href=/`
@bobbai00 bobbai00 added the bug label Sep 20, 2024
@bobbai00 bobbai00 self-assigned this Sep 20, 2024
@bobbai00 bobbai00 changed the title Update the ng build command in deploy-daemon.sh Upgrade the ng build command in deploy-daemon.sh Sep 20, 2024
@Yicong-Huang
Copy link
Contributor

Yicong-Huang commented Sep 20, 2024

Hi @bobbai00 can you add in the description which angular version is deprecating this argument, and which one we are moving to? The link you send indicates it is version 18.0.0, but we are using angular 16?

@bobbai00
Copy link
Contributor Author

Hi @bobbai00 can you add in the description which angular version is deprecating this argument, and which one we are moving to? The link you send indicates it is version 18.0.0, but we are using angular 16?

sure, I added it.

@bobbai00 bobbai00 added the fix label Sep 20, 2024
@bobbai00 bobbai00 merged commit 83c926b into master Sep 20, 2024
@bobbai00 bobbai00 deleted the jiadong-fix-deploy-daemon branch September 20, 2024 17:39
PurelyBlank pushed a commit that referenced this pull request Dec 4, 2024
This PR updates the `ng build` command in `deploy-daemon.sh`.

Before the change, the command is:
`ng build --prod  --deploy-url=/ --base-href=/`

However, `--prod` is deprecated in angular 14 according to
https://github.com/angular/angular-cli/blob/main/CHANGELOG.md#breaking-changes-2
<img width="1072" alt="Screenshot 2024-09-19 at 10 02 56 PM"
src="https://github.com/user-attachments/assets/03e5052e-d29a-4aeb-9d0c-71239795baef">

And our angular version is angular 16.

So I updated it to make it align with current angular: `ng build
--configuration production --deploy-url=/ --base-href=/`
@Yicong-Huang Yicong-Huang removed the bug label Oct 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants