Skip to content

Conversation

@JamieMagee
Copy link
Member

Error: /home/runner/work/webhooks.net/webhooks.net/src/Octokit.Webhooks.AzureFunctions/GitHubWebhooksHttpFunction.cs(54,49): error IDE0305: Collection initialization can be simplified (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0305) [/home/runner/work/webhooks.net/webhooks.net/src/Octokit.Webhooks.AzureFunctions/Octokit.Webhooks.AzureFunctions.csproj]

Error: /home/runner/work/webhooks.net/webhooks.net/src/Octokit.Webhooks.AzureFunctions/GitHubWebhooksHttpFunction.cs(54,49): error IDE0305: Collection initialization can be simplified (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0305) [/home/runner/work/webhooks.net/webhooks.net/src/Octokit.Webhooks.AzureFunctions/Octokit.Webhooks.AzureFunctions.csproj]
@github-actions
Copy link

👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labeled with Status: Up for grabs. You & others like you are the reason all of this works! So thank you & happy coding! 🚀

@JamieMagee JamieMagee enabled auto-merge (squash) February 12, 2025 23:45
@JamieMagee JamieMagee requested a review from Copilot February 12, 2025 23:48
Copy link

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.

PR Overview

This PR aims to fix the IDE0305 warning by simplifying the collection initialization in the GitHubWebhooksHttpFunction. The key change is updating the header dictionary initialization to remove an unnecessary conversion using ToArray().

Changes

File Description
src/Octokit.Webhooks.AzureFunctions/GitHubWebhooksHttpFunction.cs Simplifies header conversion to fix IDE0305 error

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Tip: Turn on automatic Copilot reviews for this repository to get quick feedback on every pull request. Learn more

@JamieMagee JamieMagee merged commit ef000c0 into main Feb 13, 2025
9 checks passed
@JamieMagee JamieMagee deleted the IDE0305 branch February 13, 2025 06:40
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