Skip to content

Comments

chore: remove unused angular-loader and angular-mocks dependencies#91

Closed
Kunal1522 wants to merge 1 commit intom-lab:mainfrom
Kunal1522:fix/remove-unused-deps
Closed

chore: remove unused angular-loader and angular-mocks dependencies#91
Kunal1522 wants to merge 1 commit intom-lab:mainfrom
Kunal1522:fix/remove-unused-deps

Conversation

@Kunal1522
Copy link
Contributor

Summary

Removes two unused dependencies from package.json that have no references anywhere in the project.

Closes #40

What was removed

Dependency Reason unused
@bower_components/angular-loader Not referenced in any <script> tag in index.html or any language variant
@bower_components/angular-mocks Intended for Karma/Jasmine unit tests, but no test files exist in this repository

Verification

Searched the entire app/ directory and gulpfile.js for any reference to either package — no matches found:

$ grep -r "angular-loader" app/
(no output)

$ grep -r "angular-mocks" app/
(no output)

All other dependencies remain — confirmed still in active use via <script> tags (e.g. angular-route, angular-gettext, jquery).

Neither angular-loader nor angular-mocks are referenced anywhere in
the project's HTML or JS files.

- angular-loader: not used by any script tag in index.html or any
  language variant

- angular-mocks: intended for unit tests with Karma/Jasmine, but
  no test files exist in this repository

Closes m-lab#40
@robertodauria
Copy link
Contributor

robertodauria commented Feb 20, 2026

Thanks for the contribution. Closing this as it is superseded by #83, which rewrites the codebase removing Angular, jQuery, Gulp, and the old app/ directory structure. The files modified by this PR either no longer exist after that change or the change has already been made.

We aim to merge #83 next week, which will give contributors a stable, modern base to work from. Thoughtful, targeted contributions that build on where the project is heading are much more valuable. We'd encourage you to review #83 and open any future PRs on top of that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove unused dependencies

2 participants