[GitHub] janpio commented on a change in pull request #199: Add previous patch dependency note

[GitBox <gi...@apache.org>]

janpio commented on a change in pull request #199: Add previous patch dependency note
URL: https://github.com/apache/cordova-coho/pull/199#discussion_r217216127
 
 

 ##########
 File path: docs/tools-release-process.md
 ##########
 @@ -53,6 +53,8 @@ This document describes how to release all tools packages at once but this is no
 
 (with `cordova-common` -> `cordova-registry-mapper` removed from `master` branch of `cordova-common`)
 
+NOTE: Whenever we start a new major release it is possible that the `master` branch of Cordova packages will have explicit dependency on previous patch release of other packages.
 
 Review comment:
   Why is this a problem? If tests are passing, this is obviously fine and there is no real need to update.
   
   If we want to keep those dependencies updated, this should be a separate step in the release process or a general guideline.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org