-
Notifications
You must be signed in to change notification settings - Fork 30
[Task]: Update Dependencies, remove resolutions if possible #304
Comments
Hi @yaldram, I am a first-time contributor. Could you assign this to me? |
@albinAppsmith can you help @ShravaniRoy here, may be she needs to work on this in the old-design system folder. |
@ShravaniRoy I would suggest you to fork the repository and start working, all the update dependencies tasks needs to be done under the |
Sure, will get started. Thank you! |
Hi @albinAppsmith ! I have forked the repo, created a branch, and upgraded the dependencies. I have also run the command "ncu -upgrade" to avoid conflicts. However, when I try to run the build script, there are multiple compatibility errors thrown. Can you help? |
@ShravaniRoy sure, can you show me the errors you are getting? |
@ShravaniRoy instead of using |
@albinAppsmith , please find the below for error: |
@ShravaniRoy Can you also share upgraded dependencies list, or package.json also works. |
Sure. "devDependencies": { |
@ShravaniRoy I think you have updated a lot of dependencies so it has become pretty hard to debug the issue. I would suggest you to only upgrade, rollup related dependencies and storyboook, make sure you uninstall the dependencies and re-install them. For storybook remove all deps using |
@ShravaniRoy |
@yaldram I have updated the |
cool yes please commit all the files and raise a pull request. I will go through your Pull Request. |
Is there an existing issue for this?
SubTasks
The text was updated successfully, but these errors were encountered: