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

[0.11.x] Fix conflicting npm dependencies and bump typescript 4 to 5 - Fixes #1182 #1241

Merged
merged 1 commit into from
Feb 3, 2024

Conversation

johnaohara
Copy link
Member

Backport: #1224

Fixes Issue #1182

Changes proposed

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.

@johnaohara johnaohara added type/enhancement An enhancement to an existing feature branch/0.11.x The 0.11.x release area/dependency Dependency Upgrade and removed type/enhancement An enhancement to an existing feature labels Feb 2, 2024
@shivarm
Copy link
Member

shivarm commented Feb 3, 2024

@johnaohara We need to regenerate package-lock.json

steps

  1. Delete old package-lock.json
  2. Run npm i in Horreum-web
  3. force push

@johnaohara
Copy link
Member Author

@johnaohara We need to regenerate package-lock.json

steps

  1. Delete old package-lock.json
  2. Run npm i in Horreum-web
  3. force push

yeah, package-lock is becoming a massive pain point for us, I need to think about ways of handling it so we are not being constantly affected by conflicts and having to rebuild

@shivarm
Copy link
Member

shivarm commented Feb 3, 2024

Ci failure look a npm issue-https://github.com/Hyperfoil/Horreum/actions/runs/7764614586/job/21178269891?pr=1241#step:6:619

see discussions and solution on the issue-npm/cli#4828 (comment)

yeah, package-lock is becoming a massive pain point for us, I need to think about ways of handling it so we are not being constantly affected by conflicts and having to rebuild

we have to stop dependabot for npm dependencies now

@shivarm
Copy link
Member

shivarm commented Feb 3, 2024

I have not access of this otherwise i can try to fix it in my env

@johnaohara
Copy link
Member Author

I don't think dependabot is the cause of this issue, I built the package-lock.json on my Mac, which is currently using npm 9.5.0

I think a lot of our problems are coming from different versions of node/npm in different environments. I think we need to consider using a consistent env (possibly CI) to build package-lock.json for us

@shivarm
Copy link
Member

shivarm commented Feb 3, 2024

I don't think dependabot is the cause of this issue, I built the package-lock.json on my Mac, which is currently using npm 9.5.0

Yeah, It is.

I think a lot of our problems are coming from different versions of node/npm in different environments. I think we need to consider using a consistent env (possibly CI) to build package-lock.json for us

Then let's keep current node/npm versions which is used by horreum, in local as well so I have this already and I think you have to follow this.

@johnaohara
Copy link
Member Author

I don't think dependabot is the cause of this issue, I built the package-lock.json on my Mac, which is currently using npm 9.5.0

Yeah, It is.

Could you explain why you think dependabot is the issue?

@johnaohara
Copy link
Member Author

@shivam-sharma7 I think we should take the package-lock.json chat to a discussion

@shivarm
Copy link
Member

shivarm commented Feb 3, 2024

Could you explain why you think dependabot is the issue?

as @jesperpedersen said-#587 (comment)

@johnaohara johnaohara merged commit 9c63ded into 0.11.x Feb 3, 2024
2 checks passed
@johnaohara johnaohara deleted the bp-0.11.x-488c433 branch February 3, 2024 06:05
@johnaohara johnaohara changed the title [0.11.x] Fix conflicting npm dependencies and bump typescript 4 to 5 [0.11.x] Fix conflicting npm dependencies and bump typescript 4 to 5 - Fixes #1182 Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependency Dependency Upgrade branch/0.11.x The 0.11.x release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants