Skip to content

Conversation

EagleoutIce
Copy link
Member

No description provided.

@EagleoutIce EagleoutIce requested a review from Copilot July 31, 2025 07:51
@EagleoutIce EagleoutIce linked an issue Jul 31, 2025 that may be closed by this pull request
16 tasks
Copy link

@Copilot 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.

Pull Request Overview

This PR introduces a new linter rule for detecting network-related functions in R code. The implementation refactors existing code to create reusable utilities for function detection patterns.

  • Adds a new "network-functions" linter rule to flag functions that perform network operations
  • Extracts common function-finding logic into a reusable utility module
  • Updates the deprecated functions rule to use the new shared utility

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/linter/rules/network-functions.ts Implements the new network functions linter rule
src/linter/rules/function-finder-util.ts Creates shared utility for function detection logic
src/linter/rules/deprecated-functions.ts Refactors to use the new shared utility
src/linter/linter-rules.ts Registers the new network functions rule
test/functionality/linter/lint-network-functions.test.ts Adds comprehensive tests for the new rule
test/functionality/linter/lint-deprecated-functions.test.ts Updates tests to use new metadata field names
Comments suppressed due to low confidence (2)

@EagleoutIce EagleoutIce marked this pull request as draft August 3, 2025 07:29
@EagleoutIce EagleoutIce force-pushed the main branch 4 times, most recently from c292b15 to 26d6e5f Compare August 20, 2025 14:36
@EagleoutIce EagleoutIce marked this pull request as ready for review September 2, 2025 21:34
@EagleoutIce EagleoutIce merged commit ba38056 into main Sep 29, 2025
19 checks passed
@EagleoutIce EagleoutIce deleted the 1813-linter-network-functions branch September 29, 2025 19:36
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.

[Linter]: Network functions
2 participants