Skip to content

Perform installation according to package manager#744

Merged
aman-v-singh merged 13 commits intodevelopfrom
deepinstall-made-package-manager-aware
May 27, 2024
Merged

Perform installation according to package manager#744
aman-v-singh merged 13 commits intodevelopfrom
deepinstall-made-package-manager-aware

Conversation

@aman-v-singh
Copy link
Contributor

Copy link
Member

@VShingala VShingala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed, let's publish beta version and test it on App to make sure it works as expected.

@patricksevat
Copy link

@opicacek could you connect with @aman-v-singh to test out the beta version?

@aman-v-singh
Copy link
Contributor Author

aman-v-singh commented May 22, 2024

I have published the beta version 1.10.1-beta.1.

@opicacek
Copy link
Member

@aman-v-singh I tried to do the upgrade locally to the version 1.10.1-beta.1, the installation was successful. In my case the yarn was used. Additional dependencies in codegens were also installed. 👍

@VShingala
Copy link
Member

@aman-v-singh Let's look into failing tests and get this merged.

@aman-v-singh
Copy link
Contributor Author

@aman-v-singh Let's look into failing tests and get this merged.

Test fixed in this PR.
#745

@robbieaverill
Copy link

Would love to see this in a tagged release - I believe it will fix an issue I am having with installing in a monorepo (using pnpm)

@robbieaverill
Copy link

@aman-v-singh would you be able to release a new beta tag, please?

@VShingala
Copy link
Member

@robbieaverill This is now available with v1.11.0. http://npmjs.com/package/postman-code-generators/v/1.11.0

@robbieaverill
Copy link

@aman-v-singh would you be able to release a new beta tag, please?

@VShingala
Copy link
Member

@robbieaverill This is now available with v1.11.0. http://npmjs.com/package/postman-code-generators/v/1.11.0

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.

5 participants