Skip to content

[ACS-11225] Remove @nx/webpack dependency#11688

Merged
DenysVuika merged 1 commit intodevelopfrom
dev-mmaliarchuk/ACS-11225-remove-@nx/webpack-dependency
Feb 26, 2026
Merged

[ACS-11225] Remove @nx/webpack dependency#11688
DenysVuika merged 1 commit intodevelopfrom
dev-mmaliarchuk/ACS-11225-remove-@nx/webpack-dependency

Conversation

@nikita-web-ua
Copy link
Contributor

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (check one with "x")

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation
  • Other... Please describe:

What is the current behaviour? (You can also link to an open issue here)

https://hyland.atlassian.net/browse/ACS-11225

What is the new behaviour?

@nx/webpack was removed as stale dependency

Does this PR introduce a breaking change? (check one with "x")

  • Yes
  • No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...

Other information:

@nikita-web-ua nikita-web-ua added the dependencies Pull requests that update a dependency file label Feb 25, 2026
Copilot AI review requested due to automatic review settings February 25, 2026 15:09
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the repository’s Node dependencies to remove the direct @nx/webpack entry, aiming to eliminate a stale Nx-related dependency from the root install set.

Changes:

  • Removed @nx/webpack from package.json dependencies.

Comment on lines 53 to 56
"@cspell/eslint-plugin": "9.4.0",
"@mat-datetimepicker/core": "15.0.2",
"@ngx-translate/core": "^16.0.4",
"@nx/webpack": "22.3.3",
"angular-oauth2-oidc": "17.0.2",
Copy link

Copilot AI Feb 25, 2026

Choose a reason for hiding this comment

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

Although @nx/webpack is removed from the root dependencies, it is still pulled in transitively via @nx/angular (per package-lock.json), so it will likely remain installed and appear in audits. If the goal is to fully eliminate @nx/webpack from the dependency tree, you’ll need to address the upstream Nx plugin dependency (e.g., via Nx package updates/removal) rather than only removing the direct entry here.

Copilot uses AI. Check for mistakes.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's expected

@nikita-web-ua nikita-web-ua force-pushed the dev-mmaliarchuk/ACS-11225-remove-@nx/webpack-dependency branch from f80aab6 to 7b2ada8 Compare February 26, 2026 15:22
@sonarqubecloud
Copy link

@DenysVuika DenysVuika merged commit 3b14351 into develop Feb 26, 2026
19 checks passed
@DenysVuika DenysVuika deleted the dev-mmaliarchuk/ACS-11225-remove-@nx/webpack-dependency branch February 26, 2026 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A/N BDU dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants