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

Version 1.0.0 release #10

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

Conversation

johns31459
Copy link
Collaborator

@johns31459 johns31459 commented Jul 23, 2024

Aligns with Go SDK (https://github.com/CrowdStrike/foundry-fn-go):

  • Cuts version 1.0.0 release.
  • Removes hard dependency on falconpy.
  • Adds convenience method crowdstrike.foundry.function.cloud() to fetch a falconpy compatible cloud identifier.
  • Adds function ID and version to inbound request.
  • Adds a Logger as a parameter to each handler.

README.md Outdated

### `falconpy`
As of version 1.0.0, `falconpy` no longer ships with Foundry Function Python.
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: drop the first two lines here. No one knows we had falconpy in this repo as the number of users is quite low. Just remove the noise and get straight to the point below. Run the risk of confusing folks here with to much context.

@@ -106,3 +105,19 @@ def call(func):
))

return call


def cloud() -> str:
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

Copy link
Collaborator

@jsteenb2 jsteenb2 left a comment

Choose a reason for hiding this comment

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

lgtm, once this is passing in our e2e test suite, gediup!

@johns31459 johns31459 force-pushed the 20240723/additional-request-params branch from 472594f to f0a6233 Compare January 2, 2025 18:33
@johns31459 johns31459 force-pushed the 20240723/additional-request-params branch from f0a6233 to ed295f5 Compare January 2, 2025 20:17
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.

2 participants