- Issue created by @Agami4
- Status changed to Needs review
7 months ago 7:54am 26 April 2024 - First commit to issue fork.
- Merge request !215Draft: Issue #3442423 - Updates deprecated packages and nvmrc to latest version. → (Closed) created by sourojeetpaul
Hi @Agamich, I'v also encountered the gulp package deprecation issue, and tried to apply your patch, but even after applying the patch some issues still persists. Also I think it'll be better to raise an MR for the same, as the community prefers MR over patches.
So I took a closer look and solved some remaining issues.
Updated the nvmrc to the latest version, also there were some issue in the .eslintrc.json due to some missing plugins, some config dependency related to the airbnb-base was still missing and looked like the config of gulp-eslint is also broken, several people reported the same, also its not that much well maintained nowadays, as a result its encounterimg some issues while building the dep graph So a better option will be to opt for the latest gulp-eslint-new plugin.
You can refer to this issue for the same: https://github.com/airbnb/javascript/issues/2545
I've incorporated all these in changes on MR!215
Please have a look!After these changes eslint is correctly referring to some ES6 Module errors, which needs to fixed. I think that's beyond the scope of this issue and perhaps we can address that after proper merger of this one.
- Issue was unassigned.
- 🇮🇳India Hritick
Hey @Agamich , I have reviewed the MR 215 and according to the issue summary after removing the old packages and re-modifying the gulp file it is working fine and all the functions are running and errors are thrown if there. Attaching screenshots for reference below.
Thanks and regards.
- Status changed to RTBC
5 months ago 1:05pm 4 July 2024 - Status changed to Postponed
3 months ago 12:12pm 28 August 2024