GivingThanks

The function doesn't adhere to Solidity's naming conventions, which generally recommend using camelCase (also called "mixedCase") for function parameters. In `function, _registry` it does not follow this convention because of the leading underscore.

​

```function updateRegistry(address _registry) public {

registry = CharityRegistry(_registry);

}

```

​

**Mitigation**

​

Parameter Naming: Changed `_registry` to `registryAddress` to follow camelCase naming.

​

```function updateRegistry(address registryAddress) public {

require(registryAddress != address(0), "Invalid address");

registry = CharityRegistry(registryAddress);

}

```

​