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

Add CLI as a function deployment type #8215

Merged
merged 4 commits into from
Jun 25, 2024
Merged

Conversation

byawitz
Copy link
Member

@byawitz byawitz commented May 29, 2024

What does this PR do?

Set the function deployment type as cli when x-sdk-language matches the cli. This will make it possible to filter function deployments by CLI.

Test Plan

Tested against local 1.5.5 instance

Related

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

@byawitz byawitz added the product / functions Fixes and upgrades for the Appwrite Functions. label May 29, 2024
@@ -1163,6 +1163,7 @@
}

$activate = (bool) filter_var($activate, FILTER_VALIDATE_BOOLEAN);
$type = $request->getHeader('x-sdk-language') === 'cli' ? 'cli' : 'manual';
Copy link
Contributor

Choose a reason for hiding this comment

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

Any chance we can update a test case to check for this?

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

Copy link
Member Author

Choose a reason for hiding this comment

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

The failing test is not relevant

@byawitz byawitz requested a review from stnguyen90 June 17, 2024 18:21
@christyjacob4 christyjacob4 merged commit c692661 into 1.6.x Jun 25, 2024
22 of 23 checks passed
@christyjacob4 christyjacob4 deleted the feat-func-deploy-type branch June 25, 2024 13:59
@stnguyen90 stnguyen90 changed the title feat: Adding cli to the function deployment type Add CLI as a function deployment type Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product / functions Fixes and upgrades for the Appwrite Functions.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants