Skip to content
This repository has been archived by the owner on Nov 23, 2021. It is now read-only.

Fixes for --skip-pre & --skip-post methods #120

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

thim81
Copy link
Contributor

@thim81 thim81 commented Aug 18, 2021

Fixes a bug that when using the --skip-pre/--skip-post options was passed, the converter did not remove the pre-request/post-request scripts at the collection or folder level.

@thim81
Copy link
Contributor Author

thim81 commented Aug 18, 2021

Linked to #105

@thim81
Copy link
Contributor Author

thim81 commented Sep 1, 2021

For those that run into the issue with the --skip-pre & --skip-post methods , I have provided a forked NPM package version that contains a fix for the issue:

Replace in your packages.json:

"dependencies": {
    "postman-to-k6": "^1.5.0"
  }

to

"dependencies": {
    "@apideck/postman-to-k6": "^1.7.0"
  }

This is a forked NPM package, to facilitate other users. The current maintainers are quite overloaded by the acquisition of K6 by Grafana. By forking the repo & package, users can keep using new PR's & (security) fixes. At a later stage the changes can potentially be merged back in the original postman-to-k6 repo.

The changelog contains the differences between the original package & the forked version.

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

Successfully merging this pull request may close these issues.

None yet

1 participant