BeanMethodsNotPublic should not change visibility of overrided methods

[murdos]

Otherwise it leads to compilation failures.

Example:

public interface AbstractConfiguration {
    MyBean mybean();
}

@Configuration
public class ConcreteConfiguration implements AbstractConfiguration {

    @Bean
    @Override
    public MyBean mybean() {
        return new MyBean();
    }

}

[sambsnyd]

Thanks for reporting this issue.
Fixing this requires on openrewrite/rewrite#150
In the meantime we should remove it from the best practices recipe