Skip to content
This repository was archived by the owner on Sep 3, 2024. It is now read-only.

APP-15551 - Updating integration to prep for integration helpers #54

Merged
merged 3 commits into from
Jul 3, 2024

Conversation

jzolo22
Copy link
Contributor

@jzolo22 jzolo22 commented Jul 3, 2024

  • Bumped sdk packages
  • Applied prettier formatting to all files
  • Fixed ts issues
    • Mostly this was just turning async .map functions into iteratee functions

@jzolo22 jzolo22 requested a review from a team as a code owner July 3, 2024 20:43
Copy link

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@ampproject/remapping@2.3.0 None 0 78.9 kB jridgewell
npm/@babel/code-frame@7.24.7 environment +1 29.7 kB nicolo-ribaudo
npm/@babel/compat-data@7.24.7 None 0 65.6 kB nicolo-ribaudo
npm/@babel/core@7.24.7 environment, filesystem, unsafe +1 814 kB nicolo-ribaudo
npm/@babel/generator@7.24.7 None +1 525 kB nicolo-ribaudo
npm/@babel/helper-compilation-targets@7.24.7 None +1 121 kB nicolo-ribaudo
npm/@babel/helper-environment-visitor@7.24.7 None 0 6.72 kB nicolo-ribaudo
npm/@babel/helper-function-name@7.24.7 None 0 21.7 kB nicolo-ribaudo
npm/@babel/helper-hoist-variables@7.24.7 None 0 6.99 kB nicolo-ribaudo
npm/@babel/helper-module-imports@7.24.7 None 0 63.7 kB nicolo-ribaudo
npm/@babel/helper-module-transforms@7.24.7 None 0 158 kB nicolo-ribaudo
npm/@babel/helper-simple-access@7.24.7 None 0 14.1 kB nicolo-ribaudo
npm/@babel/helper-split-export-declaration@7.24.7 None 0 10.8 kB nicolo-ribaudo
npm/@babel/helper-string-parser@7.24.7 None 0 31.8 kB nicolo-ribaudo
npm/@babel/helper-validator-identifier@7.24.7 None 0 49.3 kB nicolo-ribaudo
npm/@babel/helper-validator-option@7.24.7 None 0 11.8 kB nicolo-ribaudo
npm/@babel/helpers@7.24.7 None 0 859 kB nicolo-ribaudo
npm/@babel/highlight@7.24.7 environment +4 46.8 kB nicolo-ribaudo
npm/@babel/parser@7.24.7 None 0 1.89 MB nicolo-ribaudo
npm/@babel/template@7.24.7 None 0 69 kB nicolo-ribaudo
npm/@babel/traverse@7.24.7 None +1 674 kB nicolo-ribaudo
npm/@babel/types@7.24.7 environment +1 2.42 MB nicolo-ribaudo
npm/@eslint-community/eslint-utils@4.4.0 Transitive: environment, eval, filesystem, unsafe +48 8.47 MB eslint-community-bot
npm/@eslint-community/regexpp@4.11.0 None 0 446 kB eslint-community-bot
npm/@jridgewell/gen-mapping@0.3.5 None 0 81.6 kB jridgewell
npm/@jridgewell/set-array@1.2.1 None 0 17.9 kB jridgewell
npm/@jupiterone/integration-sdk-core@12.8.3 environment Transitive: eval, unsafe +9 6.31 MB jupiterone-dev
npm/@jupiterone/integration-sdk-dev-tools@12.8.3 Transitive: environment, eval, filesystem, network, shell, unsafe +524 75.9 MB jupiterone-dev
npm/@jupiterone/integration-sdk-testing@12.8.3 environment Transitive: eval, filesystem, network, shell +182 27.2 MB jupiterone-dev
npm/@nodelib/fs.scandir@2.1.5 filesystem +1 28.6 kB mrmlnc
npm/@nodelib/fs.stat@2.0.5 filesystem 0 11.8 kB mrmlnc
npm/@nodelib/fs.walk@1.2.8 None +2 63.2 kB mrmlnc
npm/@types/json-schema@7.0.15 None 0 31.7 kB types
npm/@types/semver@7.5.8 None 0 23.3 kB types
npm/@typescript-eslint/utils@6.21.0 Transitive: environment, filesystem +12 2.97 MB jameshenry
npm/browserslist@4.23.1 environment, filesystem 0 62.9 kB ai
npm/caniuse-lite@1.0.30001640 None 0 2.08 MB caniuse-lite
npm/convert-source-map@2.0.0 None 0 15.9 kB phated
npm/electron-to-chromium@1.4.816 None 0 296 kB kilianvalkhof
npm/escalade@3.1.2 filesystem 0 11.6 kB lukeed
npm/eslint-visitor-keys@3.4.3 None 0 32.3 kB eslintbot
npm/espree@9.6.1 None +1 98 kB eslintbot
npm/expect@29.7.0 Transitive: unsafe +17 940 kB simenb
npm/glob-parent@6.0.2 None 0 7.72 kB phated
npm/jest-util@29.7.0 environment +7 604 kB simenb
npm/json5@2.2.3 None 0 235 kB jordanbtucker
npm/lru-cache@5.1.1 None 0 15.7 kB isaacs
npm/node-releases@2.0.14 None 0 34 kB chicoxyzzy
npm/prettier@3.3.2 environment, filesystem, unsafe 0 8.25 MB prettier-bot
npm/pretty-format@29.7.0 Transitive: environment +4 546 kB simenb
npm/semver@7.6.2 None 0 95.4 kB npm-cli-ops
npm/undici-types@5.26.5 None 0 73.1 kB ethan_arrowood
npm/update-browserslist-db@1.1.0 environment, filesystem, shell +1 19.9 kB ai
npm/yallist@3.1.1 None 0 14.8 kB isaacs
npm/yargs-parser@21.1.1 environment, filesystem 0 128 kB oss-bot

🚮 Removed packages: npm/@ampproject/remapping@2.2.0, npm/@babel/code-frame@7.12.11, npm/@babel/compat-data@7.19.4, npm/@babel/core@7.12.10, npm/@babel/generator@7.12.11, npm/@babel/helper-compilation-targets@7.19.3, npm/@babel/helper-environment-visitor@7.18.9, npm/@babel/helper-function-name@7.12.11, npm/@babel/helper-hoist-variables@7.18.6, npm/@babel/helper-module-imports@7.12.5, npm/@babel/helper-module-transforms@7.12.1, npm/@babel/helper-simple-access@7.12.1, npm/@babel/helper-split-export-declaration@7.12.11, npm/@babel/helper-validator-identifier@7.12.11, npm/@babel/helpers@7.12.5, npm/@babel/parser@7.12.11, npm/@babel/template@7.12.7, npm/@babel/traverse@7.12.12, npm/@babel/types@7.12.12, npm/@jridgewell/gen-mapping@0.1.1, npm/@jridgewell/set-array@1.1.2, npm/@jupiterone/integration-sdk-core@9.8.1, npm/@jupiterone/integration-sdk-dev-tools@9.8.1, npm/@jupiterone/integration-sdk-testing@9.8.1, npm/@nodelib/fs.stat@2.0.4, npm/@types/json-schema@7.0.11, npm/abab@2.0.6, npm/convert-source-map@1.7.0, npm/espree@7.3.1, npm/globals@13.17.0, npm/ignore@5.2.0, npm/is-glob@4.0.1, npm/jest-matcher-utils@27.5.1, npm/jest-util@27.5.1, npm/json5@2.1.3, npm/lodash.sortby@4.7.0, npm/pretty-format@27.5.1, npm/semver@7.3.8, npm/whatwg-url@8.7.0, npm/word-wrap@1.2.3, npm/yargs-parser@20.2.4

View full report↗︎

@@ -174,7 +182,17 @@ export class ServicesClient {
{ retryAfter },
`Received a rate limit error. Waiting before retrying.`,
);
await sleep(retryAfter);
sleep(retryAfter)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Apparently handleError doesn't accept functions that return promises. Not sure how this was working with async/await before 🤔

@jzolo22
Copy link
Contributor Author

jzolo22 commented Jul 3, 2024

Will release together with createIntegrationHelper updates once those are completed

@jzolo22 jzolo22 merged commit c9291a8 into main Jul 3, 2024
8 checks passed
@jzolo22 jzolo22 deleted the APP-15551 branch July 3, 2024 20:58
@j1-internal-automation
Copy link
Collaborator

🚀 PR was released in v4.3.0 🚀

@j1-internal-automation j1-internal-automation added the released This issue/pull request has been released. label Jul 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants