search

expressjs / expressjs.com

https://expressjs.com
Other
5.23k stars 1.42k forks source link

removed unused assets. #1493

Closed MehfoozurRehman closed 2 months ago

MehfoozurRehman commented 2 months ago

according to the feedback on https://github.com/expressjs/expressjs.com/pull/1489 removed the unused assets

MehfoozurRehman commented 2 months ago

and sorry for the inconvenience with the old pull request the vs code was asking me to fork the repo I already had forked so I deleted the fork and then forked again and that fixed it.

crandmck commented 2 months ago

Thanks @MehfoozurRehman ... I checked out the fork and ran it locally to make sure there were no issues, and it LGTM.

I just want to make sure that there's no reason to keep these file around that I'm not thinking of, so I've asked for another review from @jonchurch just to be sure.