cleanup(build): some bazel related cleanup

[mmalerba]

• Removed unused dep in e2e-app
• Updates rules_sass

[devversion]

LGTM for the unused dep removal and sass update.

Not sure about the changes to the highlight.js build file. That shouldn't be necessary. I think the CI issue is related to some transitive dependency issue that is caused by a change in the TypeScript rules.

[devversion]

Dashboard? 🤔

[mmalerba]

Yeah I don't know where this came from, there's nothing under tools/dashboard other than node_modules directories.

I was seeing some issues when running bazel build ... that seemed to be fixed by the highlight.js stuff I added

[mmalerba]

#15649 resolved the highlight.js issue, reverted the change in this PR

[devversion]

LGTM. I think we should remove the dashboard stuff as there is no dashboard folder in the repo. This probably showed up because node_modules are git-ignored and you installed the deps of the dashboard back when it still existed?

[mmalerba]

ahh that explains it, will fix

[angular-automatic-lock-bot]

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.}