-
-
Notifications
You must be signed in to change notification settings - Fork 492
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build release procedure updates to include docs in release tag #7412
Conversation
c1dbcc7
to
626dc19
Compare
The alternative to creating a changelog when tagging a new release; is to create one just after each release is made (for the upcoming release). In this approach the docs would have a changelog visible for the upcoming release; it could have a note at the top indicating when the release was scheduled for.
|
This can be back ported to 4.2.x and 3.12.x |
This may need to be rebased and tested again; please let's review these build changes before they bit-rot :) |
626dc19
to
22c46c6
Compare
@josegar74 I have updated release example version 4.4.1 latest release. |
@josegar74 this is the PR that I would like to see included, and troubleshooted during your release process. |
Signed-off-by: Jody Garnett <[email protected]>
# Conflicts: # docs/manual/docs/contributing/doing-a-release.md
Signed-off-by: Jody Garnett <[email protected]>
Signed-off-by: Jody Garnett <[email protected]>
Signed-off-by: Jody Garnett <[email protected]>
b677c42
to
93edc60
Compare
I have recreated this PR from main resulting in a clean commit history |
Signed-off-by: Jody Garnett <[email protected]>
@@ -198,6 +238,14 @@ Once the release branch has been thoroughly tested and is stable a release can b | |||
</list> | |||
</entry> | |||
``` | |||
|
|||
Update documentation to reflect series change of `latest`, `stable`, `maintenance` and `archive`: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure about this, is not the same as described in https://github.com/geonetwork/core-geonetwork/pull/7412/files#diff-afba6fb6c27ca0266f6c91620aac83e1f176672301fd1c3236755cd435247617R80 ?
pom.xml
Outdated
@@ -1395,6 +1389,7 @@ | |||
<module>slave</module> | |||
<module>schemas-test</module> | |||
<module>web-ui</module> | |||
<module>docs</module> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably I'm misunderstanding about including docs in the release tag. Is the intention to include the docs in the war
file?
I tried:
mvn clean install -DskipTests -Pwar -Pwro4j-prebuild-cache
And seems no docs included, tried also with -Pall
profile, same results.
frombranch=origin/main | ||
versionbranch=4.2.x | ||
series=4.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe can be good to use 4.4 for example instead of 4.2 (see other related variables below)
The backport to
stderr
stdout
To backport manually, run these commands in your terminal: # Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-4.2.x 4.2.x
# Navigate to the new working tree
cd .worktrees/backport-4.2.x
# Create a new branch
git switch --create backport-7412-to-4.2.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick 5890c43c0d767eea4c27d47fb64e0f894a1f7262,caa9e32c1eae9f44b78f873f8c2fcbdf765191cf,5e340d13ce1a6bb8258e7abcc427ef908ec86e24,fd30bc92111d815227d74680f3532934fdc90de8,a5032d901afb18cb7499dd10acd3111ce60d724e,16b9737d41ca42329c1baf8a9ad81f593b038244,88e1942f11d29f32b2f49808561e74158dbcf8eb,93edc607756f6b21e5fe15303dd12eb263ba1219,cd6986b86eee3ec8b86bf8533f862c86922af3bd
# Push it to GitHub
git push --set-upstream origin backport-7412-to-4.2.x
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-4.2.x Then, create a pull request where the |
…7412) * move changes to their own folder Signed-off-by: Jody Garnett <[email protected]> * update doing a release for doc updates prior to tagging release * automation used to deploy manual, manual deploy during release not longer required * changelog management using git notes to document major features * housekeeping: clean up commented out pom.xml snippet Signed-off-by: Jody Garnett <[email protected]> * update docs/README.md description to reflect contents Signed-off-by: Jody Garnett <[email protected]> * gitIgnore WEB-INF schemapublication folder Signed-off-by: Jody Garnett <[email protected]> * Change to 4.4.1 release example * Update build procedure to deploy datastorages Signed-off-by: Jody Garnett <[email protected]> --------- Signed-off-by: Jody Garnett <[email protected]>
Updated the build release instructions to cover the creation of new markdown changelog page.
This needs to occur earlier in the release process so that the documentation is included in the release tag.
Also noted that the docs were built after the war, and thus were not including the online help when requested with the
docs
profile.See also: #7411 to publish docs when a release is tagged.
change log creation
Please see doing a release.md for proccess of using
git note
in the creation of a change log.👍🏻 This approach allows pull requests to be back ported, with an optional "change log" associated with the squashed commit. This way the change log when generated for each branch includes the appropriate information.
👎🏻 Keep in mind that git notes, like git tags, are shared between all repositories (and do not really offer a smooth conflict resolution mechanism).