-
-
Notifications
You must be signed in to change notification settings - Fork 147
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
chore(deps): upgrade @nestjs to 9.3.9 #693
Conversation
@bahrmichael thanks for the PR. The build failed. Can you pleas take a look when you've time? |
@wing328 I added a TS instruction, assuming TS became more strict over time and that the previous build failure is not a problem. |
can we make it less strict for the time being so that the build passes? what about updating the build to make the tests pass instead? |
I'm not sure why the build fails sometimes, and I don't have permission to rerun it. It succeeds locally when I run it with node@16.19.1. Seeing the renovate changes where it sometimes failed and sometimes succeeds it feels like this is out of my control.
I couldn't find any information how to do that. TS becoming more strict is just a guess, that might be related to other upgrades. Reverting them is not a good idea just to not have the ts-ignore line. |
Why not just cast it to any if you are going to ignore the error either way? (error.response.data as any).on('data', data => this.logger.log(data.toString('utf8'))); |
Good idea! I added this change :) |
Could someone please approve the workflow. This PR upgrades a dep with a security issue. |
@wing328 friendly bump :) |
approved the workflow to run. let's see how that goes. |
@wing328 this is the second issue, that I see idle for a while and blocked me. Can we work on the community management, so that more members look over issues more frequently? This project is also used in business contexts but the speed of issue management almost holds me back from promoting it any further in sensitive environments. |
For the time being: This dependably PR seems to build smoothly: #694 A lot of people are waiting on it. |
The best way to make certain issues a higher priority is via sponsorship: https://opencollective.com/openapi_generator |
I added a revert, and things work fine locally. I'm going to unsubscribe from this issue and trust that you will resolve the security issue soon. For anyone who doesn't want to wait for this PR, you can use overrides in the package.json.
|
now all tests passed as indicated in ae627d1 thanks for the PR. |
FYI. Merged #699 in order for the build/release to pass in the master. |
@bahrmichael thanks again for the PR 🙏 I triggered a release and when I tried to install the latest openapi-generator-cli with
Is that something you can help us with by filing a PR to address these warnings? |
@wiesnery if you want to, you can now sponsor https://github.com/sponsors/kay-schecker (the author of openapi-generator-cli) directly to financially support his awesome open-source work. |
This PR upgrades the @nestjs libraries which were previously at 8.4.4 to 9.3.9.