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

Update oazapfts to latest version #4461

Closed
wants to merge 2 commits into from
Closed

Conversation

yamilelias
Copy link

Context

This is a follow-up of this PR: #4198

I've found some issues on the @rtk-query/codegen-openapi that point to dependency issues. Here are some of them:

From version 6.0.2, which was merged yesterday to version 6.1.0, there are several more fixes added to oazapfts. These are a few:

List of releases changelog:

  • support moduleResolution: node
  • codegen: upgrade dependencies
  • do not treat numeric values in multipart as blob
  • codegen: handle negative numbers correctly
  • support boolean schemas

I was looking at the issue on the pipeline when you tried to publish the package. There are issues with the cache. Rerunning the pipeline could help. Here is a reference on this issue: yarnpkg/berry#2948

What is done in this PR?

  • Updating oazapfts package to the latest version to fix issues
  • Rollback SimonEggert update to the package module to test everything is working based on the package update

Having this package on version 6.0.2 would not solve
all the issues on this dependency.
Copy link

codesandbox bot commented Jun 14, 2024

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@yamilelias
Copy link
Author

@markerikson this PR is intended to help you to fix the publication of the update on oazapfts merged yesterday.

Copy link

netlify bot commented Jun 14, 2024

Deploy Preview for redux-starter-kit-docs ready!

Name Link
🔨 Latest commit 4276ff2
🔍 Latest deploy log https://app.netlify.com/sites/redux-starter-kit-docs/deploys/667c33ff388bc00008dfd84d
😎 Deploy Preview https://deploy-preview-4461--redux-starter-kit-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@markerikson
Copy link
Collaborator

Ended up creating a new PR since yours wasn't on a separate feature branch in your repo, and I think that would prevent me from rebasing it. Superseded by #4588 . Thanks!

@markerikson
Copy link
Collaborator

Should be live in https://github.com/reduxjs/redux-toolkit/releases/tag/%40rtk-query%2Fcodegen-openapi%402.0.0-alpha.0 ! Please try it out and let us know if it works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants