Skip to content

Conversation

colin-higgins
Copy link
Member

@colin-higgins colin-higgins commented Apr 2, 2021

Allows us to use dependabot for vendored dependencies.

🍯 🐝 🐝 🐝

@DataDog/apm-dotnet

@colin-higgins colin-higgins self-assigned this Apr 2, 2021
@colin-higgins colin-higgins requested a review from a team as a code owner April 2, 2021 15:50
@@ -0,0 +1,7 @@
version: 1
Copy link
Contributor

Choose a reason for hiding this comment

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

Silly question, is it okay that it's not config.yml? I don't know how sensitive the tooling is to the extension

Copy link
Member Author

Choose a reason for hiding this comment

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

Not a silly question, just an oversight on my part, thanks.
I don't know if it would work or not actually.

Copy link
Contributor

@zacharycmontoya zacharycmontoya left a comment

Choose a reason for hiding this comment

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

LGTM


<PropertyGroup>
<TargetFrameworks>netcoreapp3.1</TargetFrameworks>
<RootNamespace>Datadog.Dependabot.Honeypot</RootNamespace>
Copy link
Member

Choose a reason for hiding this comment

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

Why Honeypot?

Copy link
Member Author

@colin-higgins colin-higgins Apr 2, 2021

Choose a reason for hiding this comment

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

It seemed like an apt name.
https://en.wikipedia.org/wiki/Honeypot_(computing)
It's not an attacker, but it isn't a REAL project: It's a fake project designed to attract the dependabot.

🍯 🐝 🐝 🐝

Copy link
Member

Choose a reason for hiding this comment

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

designed to attract the dependabot.

Ah, I didn't think of it that way. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants