Skip to content

Fix function naming: anonmyze → anonymize, input → value#114

Merged
gsanchietti merged 2 commits intoissue1512from
copilot/sub-pr-113
Feb 11, 2026
Merged

Fix function naming: anonmyze → anonymize, input → value#114
gsanchietti merged 2 commits intoissue1512from
copilot/sub-pr-113

Conversation

Copy link
Copy Markdown

Copilot AI commented Feb 11, 2026

Addresses review feedback on the anonmyze helper function introduced in #113.

Changes:

  • Corrected function name from anonmyze to anonymize
  • Renamed parameter input to value to avoid shadowing Python's built-in input() function
  • Updated all 7 call sites

Before:

def anonmyze(input, uci: EUci):
    if fact_subscription_status(uci).get('status', 'no') != "no":
        return input
    h = hashlib.sha1(input.encode()).hexdigest()
    return f"anon-{h[:16]}"

After:

def anonymize(value, uci: EUci):
    if fact_subscription_status(uci).get('status', 'no') != "no":
        return value
    h = hashlib.sha1(value.encode()).hexdigest()
    return f"anon-{h[:16]}"

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: gsanchietti <804596+gsanchietti@users.noreply.github.com>
Copilot AI changed the title [WIP] Update inventory with extra info based on review Fix function naming: anonmyze → anonymize, input → value Feb 11, 2026
Copilot AI requested a review from gsanchietti February 11, 2026 10:23
@gsanchietti gsanchietti marked this pull request as ready for review February 11, 2026 10:24
@gsanchietti gsanchietti merged commit 9be43cb into issue1512 Feb 11, 2026
2 checks passed
@gsanchietti gsanchietti deleted the copilot/sub-pr-113 branch February 11, 2026 10:24
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