willholley commented on PR #106:
URL: https://github.com/apache/couchdb-helm/pull/106#issuecomment-1375847800

   The changes look good. However, I think it would be a breaking change for 
anybody that followed the instructions at 
https://github.com/apache/couchdb-helm/tree/main/couchdb#installing-the-chart 
to create their own admin secret containing a `password.ini` key. Probably this 
is a very small number of users, but it strictly should warrant a major chart 
version bump given Helm assumes [SemVer 2](https://semver.org/).
   
   Sorry not to have twigged this sooner - do you think you could bump the 
version to `4.0.0` and add a brief migration section under the version in 
`NEWS.md`?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to