Skip to content

Conversation

@patricijabrecko
Copy link
Contributor

@patricijabrecko patricijabrecko commented Jun 20, 2025

Description

  • Update runner.ts to match the one in Airdrop template
  • Update reference documentation airdrop_sdk_library_documentation.md to match the one in Airdrop SDK
  • Bump Airdrop SDK library to 1.5.1 in package.json and update devrev-snapin-template.plain accordingly

Work item

https://app.devrev.ai/devrev/works/ISS-186013

Copilot AI review requested due to automatic review settings June 20, 2025 10:36
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Airdrop SDK version and synchronizes code and documentation with the Airdrop template and SDK. Key changes include:

  • Updating version numbers and dependency references in devrev-snapin-template.plain and package.json.
  • Revising the runner.ts file to include updated error handling and operation invocation logic.
  • Updating documentation references to align with the latest SDK version.

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File Description
devrev-snapin-template.plain Bump Airdrop SDK Library version from "1.4.2" to "1.5.1".
base_folder/test/runner.ts Update runner with new disclaimer, operation handling, and post-run logic.
base_folder/package.json Update package name and dependency versions including "@devrev/ts-adaas" and adding "@devrev/typescript-sdk".

patricijabrecko and others added 2 commits June 20, 2025 12:41
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@dusano
Copy link
Contributor

dusano commented Jun 21, 2025

@patricijabrecko I've reviewed the changes. They are all good. We'll use this version of the shared folder from now on.

Copy link

@radovanjorgic radovanjorgic left a comment

Choose a reason for hiding this comment

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

Will approve for now, but let's discuss more in detail next week

@patricijabrecko patricijabrecko merged commit 245b010 into main Jun 24, 2025
2 checks passed
@patricijabrecko patricijabrecko deleted the update-docs branch June 24, 2025 08:57
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.

4 participants