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

Allow pm.require API to import packages in sandbox #1377

Merged
merged 6 commits into from
Feb 28, 2024

Conversation

coditva
Copy link
Member

@coditva coditva commented Feb 16, 2024

Allow importing packages using pm.require inside sandbox. The packages are kept as a reference inside packages key in the collection and are resolved before the script is executed.

@coditva coditva changed the title Add option to pass files used in pm.require Allow pm.require API to import packages in sandbox Feb 21, 2024
@coditva coditva force-pushed the feat/external-scripts-prefetch branch from c87506c to a148150 Compare February 21, 2024 09:41
@codecov-commenter
Copy link

codecov-commenter commented Feb 23, 2024

Codecov Report

Attention: Patch coverage is 94.23077% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 79.09%. Comparing base (bb1bf4d) to head (ed5d921).

Files Patch % Lines
lib/runner/extensions/event.command.js 94.23% 3 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1377      +/-   ##
===========================================
+ Coverage    79.05%   79.09%   +0.03%     
===========================================
  Files           45       45              
  Lines         3356     3367      +11     
  Branches       965      970       +5     
===========================================
+ Hits          2653     2663      +10     
- Misses         703      704       +1     
Flag Coverage Δ
integration 70.50% <94.23%> (+0.06%) ⬆️
legacy 46.03% <86.53%> (+0.05%) ⬆️
unit 41.75% <0.00%> (-0.14%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@coditva coditva force-pushed the feat/external-scripts-prefetch branch from 61e150d to ed5d921 Compare February 26, 2024 11:48
@coditva coditva force-pushed the feat/external-scripts-prefetch branch from 005d878 to 522f47c Compare February 28, 2024 09:39
@coditva coditva marked this pull request as ready for review February 28, 2024 09:47
@codenirvana codenirvana merged commit bca5778 into develop Feb 28, 2024
9 checks passed
@codenirvana codenirvana deleted the feat/external-scripts-prefetch branch February 28, 2024 11:28
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.

3 participants