Skip to content

Conversation

@duncanpharvey
Copy link
Collaborator

What does this PR do?

Adds support for Java and .NET Lambda Functions.

Motivation

Wider support for runtimes.

https://datadoghq.atlassian.net/browse/SVLS-4849

Additional Notes

Minor updates to readmes so variables are consistent across examples and match what is expected to be passed.

Describe how to test/QA your changes

Follow the instructions using one of the examples to deploy a Lambda function instrumented with Datadog to AWS.

target
bin
obj
.DS_Store

Choose a reason for hiding this comment

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

You could also add .DS_Store to your global git ignore file at ~/.gitignore. Here's what's in mine

storage
compile-cache
dev
.npm
.node-gyp
coverage.out

node_modules/

.bundle

.vim/.netrwhist

# Suggested from atom's gitignore
# https://github.com/atom/atom/blob/17a31e3a3729070768f31bbce7ce9bcc09f5a2b8/dot-atom/.gitignore
.atom/blob-store
.atom/compile-cache
.atom/dev
.atom/storage
.atom/.apm
.atom/.node-gyp
.atom/.npm
.atom/.atom-socket-secret-*

# https://github.com/github/gitignore/blob/main/Global/VisualStudioCode.gitignore
.vscode/*
!.vscode/settings.json
!.vscode/tasks.json
!.vscode/launch.json
!.vscode/extensions.json
!.vscode/*.code-snippets

# Local History for Visual Studio Code
.history/

# Built Visual Studio Code Extensions
*.vsix

.tmux/resurrect/

# kernprof output
*.lprof

# copilot
.vim/pack/github/start/copilot.vim

Choose a reason for hiding this comment

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

Huh, funny that I don't have .DS_Store in there....

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

TIL you can configure global .gitignore files!

@duncanpharvey duncanpharvey merged commit b2f6c39 into main Jul 1, 2024
@duncanpharvey duncanpharvey deleted the duncan-harvey/java-dotnet-support branch July 1, 2024 17:11
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