fix: exception while running command ng deploy (#2088)

[ngdevelop-tech]

Checklist

• Issue number for this PR: Exception while running command: ng run x:deploy #2088(required)
• Docs included?: (yes/no; required for all API/functional changes)
• Test units included?: (yes/no; required)
• In a clean directory, yarn install, yarn test run successfully? (yes/no; required)

Description

When we execute ng deploy command, workspace.root gives the wrong path in the Windows environment, that's why it doesn't work in the Windows environment.
Using only project root In getFirebaseProjectName and deploy, will successfully allow ng deploy while executing it at project root in windows and linux environment.
fixes #2088

[ngdevelop-tech]

@NothingEverHappens, Thanks for the approval.
Please help me, What should I do further to resolve failed build?
Do I need to merge the master branch again? ,
In Logs it is showing one log for version mismatch.

[NothingEverHappens]

@ngdevelop-tech
My understanding is that this is a build issue, and not PR issue, but I didn't look very deep,
I'm going to take a closer look next week.

Meanwhile you can try running ./node_modules/.bin/karma start --single-run --browsers ChromeHeadlessTravis --reporters mocha && yarn test:node && node tools/run-typings-test.js && bash ./test/ng-build/build.sh" on your machine to verify that it works locally

[NothingEverHappens]

After testing it locally yesterday I'm not convinced this is the best solution.

I think the culprit is the normalize function which we should use in a platform aware way.

[ngdevelop-tech]

After testing it locally yesterday I'm not convinced this is the best solution.

I think the culprit is the normalize function which we should use in a platform aware way.

@NothingEverHappens Sorry for the delayed reply, I will look into solving this with a better approach.

[Splaktar]

Is this PR no longer needed now that PR #2279 has been merged to fix #2088?

[ngdevelop-tech]

@Splaktar because PR #2279 has been merged and closes the #2088, closing this PR.