Skip to content
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

DPL-966 Replace yarn with npm as yarn is no longer required and is causing installation problems #271

Closed
2 tasks done
Sangeetha-Bheeman opened this issue Oct 30, 2023 · 1 comment · Fixed by #272 or #275
Closed
2 tasks done
Assignees
Labels
size:s Small - low effort & risk Technical debt Technical Debt Vue 3 Vue 3 upgrade

Comments

@Sangeetha-Bheeman
Copy link
Contributor

Sangeetha-Bheeman commented Oct 30, 2023

Describe the Housekeeping
As developers we should be able to install dependencies without any local installation problems and to keep it in line with other applications.

Who are the primary contacts for this story
Steve, Sangeetha

Who is the nominated tester for UAT
PSD

Acceptance criteria
To be considered successful the solution must allow.

  • To install dependencies locally without causing any errors/problems to developers.
  • The application to continue to work as expected with no loss of service for users.
@Sangeetha-Bheeman Sangeetha-Bheeman added size:s Small - low effort & risk Vue 3 Vue 3 upgrade labels Oct 30, 2023
@Sangeetha-Bheeman Sangeetha-Bheeman self-assigned this Oct 30, 2023
@Sangeetha-Bheeman Sangeetha-Bheeman changed the title DPL-nnn Replace yarn with npm DPL-966 Replace yarn with npm Oct 30, 2023
@Sangeetha-Bheeman
Copy link
Contributor Author

Sangeetha-Bheeman commented Oct 30, 2023

Tasks: To replace yarn with npm,

  • Remove yarn.lock file
  • Remove yarn commands from .gitignore and .prettierignore
  • Replace yarn commands with npm in package.json for running scripts, if applicable.
  • Replace yarn build with npm run build in compile-build.sh
  • Change README.md, replace with the appropriate npm commands
  • Replace yarn with npm commands in the files .github/workflows/js_test.yml, .github/workflows/automated_release_and_build.yml, .github/workflows/lint.yml and .github/workflows/manual_build.yml
  • Run npm install
  • Ensure all tests pass
  • Ensure CI passed and creates a release build
  • Deploy to UAT

@Sangeetha-Bheeman Sangeetha-Bheeman changed the title DPL-966 Replace yarn with npm DPL-966 Replace yarn with npm as part of Vue 3 upgrade process Oct 30, 2023
@Sangeetha-Bheeman Sangeetha-Bheeman changed the title DPL-966 Replace yarn with npm as part of Vue 3 upgrade process DPL-966 Replace yarn with npm as yarn is no longer required and is causing installation problems Oct 30, 2023
@Sangeetha-Bheeman Sangeetha-Bheeman linked a pull request Nov 7, 2023 that will close this issue
@SujitDey2022 SujitDey2022 added the Technical debt Technical Debt label Nov 9, 2023
@Sangeetha-Bheeman Sangeetha-Bheeman linked a pull request Nov 23, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:s Small - low effort & risk Technical debt Technical Debt Vue 3 Vue 3 upgrade
Projects
None yet
2 participants