Improve default parameter naming for Interceptor

[hgraceb]

Improve default parameter naming for Interceptor.

class Interceptor {
  /// The constructor only helps sub-classes to inherit from.
  /// Do not use it directly.
  const Interceptor();

  /// Called when the request is about to be sent.
  void onRequest(
    RequestOptions options,
    RequestInterceptorHandler handler,
  ) {
    handler.next(options);
  }

  /// Called when the response is about to be resolved.
  void onResponse(
    Response response,
    ResponseInterceptorHandler handler,
  ) {
    handler.next(response);
  }

  /// Called when an exception was occurred during the request.
  void onError(
    DioException err,
    ErrorInterceptorHandler handler,
  ) {
    handler.next(err);
  }
}

New Pull Request Checklist

• I have read the Documentation
• I have searched for a similar pull request in the project and found none
• I have updated this branch with the latest main branch to avoid conflicts (via merge from master or rebase)
• I have added the required tests to prove the fix/feature I'm adding
• I have updated the documentation (if necessary)
• I have run the tests without failures
• I have updated the CHANGELOG.md in the corresponding package

Additional context and info (if any)

[AlexV525]

This would breaks downstream packages' CI since it'll raise warnings about the mismatch naming AFAIK. But it's easy to change the name. Any opinion? @cfug-dev

The PR still lacks of the CHANGELOG update.

[hgraceb]

This would breaks downstream packages' CI since it'll raise warnings about the mismatch naming AFAIK. But it's easy to change the name. Any opinion? @cfug-dev

The PR still lacks of the CHANGELOG update.

Based on my knowledge, the main purpose of parameter naming in aliases is to facilitate code completion in IDEs. It doesn't have any practical impact on other code. Matching names are only required when overriding methods, so I haven't updated the CHANGELOG.

Alternatively, should I update my understanding?

[AlexV525]

Based on my knowledge, the main purpose of parameter naming in aliases is to facilitate code completion in IDEs. It doesn't have any practical impact on other code. Matching names are only required when overriding methods, so I haven't updated the CHANGELOG.

This makes sense. Thanks.