-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[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: