Merge branch 'master' into bugfix/85-Wrong-type-used-in-chapters-definition

# Conflicts:
#	.github/workflows/release_draft.yml
#	examples/release_draft.yml
#	release_notes_generator/action_inputs.py
#	release_notes_generator/utils/pull_reuqest_utils.py
#	requirements.txt
#	tests/test_action_inputs.py

Author: miroslavpojer

.github/pull_request_template.md

@@ -0,0 +1,20 @@
+## Release Notes:
+> Provide a concise summary of changes for the release notes. Include any new features, bug fixes, or other significant updates. Keep it brief and to the point.
+
+## Checklist:
+> Ensure the following tasks are completed before submission.
+- [ ] My code follows the project style guidelines.
+- [ ] I have performed a self-review of my code.
+- [ ] I have commented my code, particularly in hard-to-understand areas.
+- [ ] I have added tests that prove my fix is effective or my feature works.
+- [ ] New and existing tests pass locally with my changes.
+- [ ] I have used a code formatting tool to ensure consistent style.
+- [ ] I have run a linter to check for code quality issues.
+
+## Additional Context:
+> Add any other context about the pull request here.
+
+---
+> **Note:** Remember to link this PR to the related issue by adding `Fixes #issue_number` to the description above.
+
+Fixes **or** Closes #(issue)

.github/workflows/release_draft.yml

@@ -21,6 +21,9 @@ on:
       tag-name:
         description: 'Name of git tag to be created, and then draft release created. Syntax: "v[0-9]+.[0-9]+.[0-9]+".'
         required: true
+      from-tag-name:
+        description: 'Name of the git tag from which to detect changes from. Default value: latest tag. Syntax: "v[0-9]+.[0-9]+.[0-9]+".'
+        required: false
 
 jobs:
   release-draft:
@@ -37,13 +40,24 @@ jobs:
 
       - name: Check format of received tag
         id: check-version-tag
-        uses: AbsaOSS/version-tag-check@v0.2.0
+        uses: AbsaOSS/version-tag-check@v0.3.0
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         with:
             github-repository: ${{ github.repository }}
             version-tag: ${{ github.event.inputs.tag-name }}
 
+      - name: Check format of received from tag
+        if: ${{ github.event.inputs.from-tag-name }}
+        id: check-version-from-tag
+        uses: AbsaOSS/version-tag-check@v0.3.0
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        with:
+          github-repository: ${{ github.repository }}
+          version-tag: ${{ github.event.inputs.from-tag-name }}
+          should-exist: true
+
       - name: Generate Release Notes
         id: generate_release_notes
         uses: AbsaOSS/generate-release-notes@master

action.yml

@@ -28,6 +28,10 @@ inputs:
     description: 'Allow duplicity of issue lines in chapters. Scopes: custom, service, both, none.'
     required: false
     default: 'both'
+  from-tag-name:
+    description: 'The tag name of the previous release to use as a start reference point for the current release notes.'
+    required: false
+    default: ''
   duplicity-icon:
     description: 'Icon to be used for duplicity warning. Icon is placed before the record line.'
     required: false
@@ -116,6 +120,7 @@ runs:
         INPUT_GITHUB_TOKEN: ${{ env.GITHUB_TOKEN }}
         INPUT_TAG_NAME: ${{ inputs.tag-name }}
         INPUT_CHAPTERS: ${{ inputs.chapters }}
+        INPUT_FROM_TAG_NAME: ${{ inputs.from-tag-name }}
         INPUT_DUPLICITY_SCOPE: ${{ inputs.duplicity-scope }}
         INPUT_DUPLICITY_ICON: ${{ inputs.duplicity-icon }}
         INPUT_WARNINGS: ${{ inputs.warnings }}

examples/release_draft.yml

@@ -5,6 +5,9 @@ on:
       tag-name:
         description: 'Name of git tag to be created, and then draft release created. Syntax: "v[0-9]+.[0-9]+.[0-9]+".'
         required: true
+      from-tag-name:
+        description: 'Name of the git tag from which to detect changes from. Default value: latest tag. Syntax: "v[0-9]+.[0-9]+.[0-9]+".'
+        required: false
 
 jobs:
   release-draft:
@@ -23,14 +26,24 @@ jobs:
 
       - name: Check format of received tag
         id: check-version-tag
-        uses: AbsaOSS/version-tag-check@v0.1.0
+        uses: AbsaOSS/version-tag-check@v0.3.0
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         with:
           github-repository: ${{ github.repository }}
-          branch: 'master'
           version-tag: ${{ github.event.inputs.tag-name }}
 
+      - name: Check format of received from tag
+        if: ${{ github.event.inputs.from-tag-name }}
+        id: check-version-from-tag
+        uses: AbsaOSS/version-tag-check@v0.3.0
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        with:
+          github-repository: ${{ github.repository }}
+          version-tag: ${{ github.event.inputs.from-tag-name }}
+          should-exist: true
+
       - name: Generate Release Notes
         id: generate_release_notes
         uses: AbsaOSS/generate-release-notes@v0.3.0

release_notes_generator/action_inputs.py

@@ -46,6 +46,7 @@
     RELEASE_NOTES_TITLE,
     RELEASE_NOTE_TITLE_DEFAULT,
     SUPPORTED_ROW_FORMAT_KEYS,
+    FROM_TAG_NAME,
 )
 from release_notes_generator.utils.enums import DuplicityScopeEnum
 from release_notes_generator.utils.gh_action import get_action_input
@@ -84,6 +85,21 @@ def get_tag_name() -> str:
         """
         return get_action_input(TAG_NAME)
 
+    @staticmethod
+    def get_from_tag_name() -> str:
+        """
+        Get the from-tag name from the action inputs.
+        """
+        return get_action_input(FROM_TAG_NAME, default="")
+
+    @staticmethod
+    def is_from_tag_name_defined() -> bool:
+        """
+        Check if the from-tag name is defined in the action inputs.
+        """
+        value = ActionInputs.get_from_tag_name()
+        return value.strip() != ""
+
     @staticmethod
     def get_chapters() -> Optional[list[dict[str, str]]]:
         """
@@ -136,8 +152,9 @@ def get_skip_release_notes_labels() -> str:
         """
         Get the skip release notes label from the action inputs.
         """
-        user_choice = [item.strip() for item in get_action_input(SKIP_RELEASE_NOTES_LABELS, "").split(",")]
-        if len(user_choice) > 0:
+        user_input = get_action_input(SKIP_RELEASE_NOTES_LABELS, "")
+        user_choice = [item.strip() for item in user_input.split(",")] if user_input else []
+        if user_choice:
             return user_choice
         return ["skip-release-notes"]
 
@@ -238,6 +255,10 @@ def validate_inputs() -> None:
         if not isinstance(tag_name, str) or not tag_name.strip():
             errors.append("Tag name must be a non-empty string.")
 
+        from_tag_name = ActionInputs.get_from_tag_name()
+        if not isinstance(from_tag_name, str):
+            errors.append("From tag name must be a string.")
+
         chapters = ActionInputs.get_chapters()
         if chapters is None:
             errors.append("Chapters must be a valid yaml array.")

release_notes_generator/builder.py

@@ -29,7 +29,6 @@
 logger = logging.getLogger(__name__)
 
 
-# TODO - reduce to function only after implementing the features. Will be supported more build ways?
 # pylint: disable=too-few-public-methods
 class ReleaseNotesBuilder:
     """

release_notes_generator/generator.py

@@ -20,20 +20,24 @@
 """
 
 import logging
+import sys
 
 from typing import Optional
+import semver
+
 from github import Github
+from github.GitRelease import GitRelease
+from github.Repository import Repository
 
+from release_notes_generator.action_inputs import ActionInputs
+from release_notes_generator.builder import ReleaseNotesBuilder
 from release_notes_generator.model.custom_chapters import CustomChapters
 from release_notes_generator.model.record import Record
-from release_notes_generator.builder import ReleaseNotesBuilder
 from release_notes_generator.record.record_factory import RecordFactory
-from release_notes_generator.action_inputs import ActionInputs
 from release_notes_generator.utils.constants import ISSUE_STATE_ALL
-
 from release_notes_generator.utils.decorators import safe_call_decorator
-from release_notes_generator.utils.utils import get_change_url
 from release_notes_generator.utils.github_rate_limiter import GithubRateLimiter
+from release_notes_generator.utils.utils import get_change_url
 
 logger = logging.getLogger(__name__)
 
@@ -72,15 +76,13 @@ def generate(self) -> Optional[str]:
 
         @return: The generated release notes as a string, or None if the repository could not be found.
         """
+        # get the repository
         repo = self._safe_call(self.github_instance.get_repo)(ActionInputs.get_github_repository())
         if repo is None:
             return None
 
-        rls = self._safe_call(repo.get_latest_release)()
-        if rls is None:
-            logger.info("Latest release not found for %s. 1st release for repository!", repo.full_name)
-        else:
-            logger.debug("RLS created_at: %s, published_at: %s", rls.created_at, rls.published_at)
+        # get the latest release
+        rls: GitRelease = self.get_latest_release(repo)
 
         # default is repository creation date if no releases OR created_at of latest release
         since = rls.created_at if rls else repo.created_at
@@ -97,12 +99,12 @@ def generate(self) -> Optional[str]:
 
             # filter out closed Issues before the date
             issues = list(
-                filter(lambda issue: issue.closed_at is not None and issue.closed_at > since, list(issues_all))
+                filter(lambda issue: issue.closed_at is not None and issue.closed_at >= since, list(issues_all))
             )
             logger.debug("Count of issues reduced from %d to %d", len(list(issues_all)), len(issues))
 
             # filter out merged PRs and commits before the date
-            pulls = list(filter(lambda pull: pull.merged_at is not None and pull.merged_at > since, list(pulls_all)))
+            pulls = list(filter(lambda pull: pull.merged_at is not None and pull.merged_at >= since, list(pulls_all)))
             logger.debug("Count of pulls reduced from %d to %d", len(list(pulls_all)), len(pulls))
 
             commits = list(filter(lambda commit: commit.commit.author.date > since, list(commits_all)))
@@ -125,3 +127,59 @@ def generate(self) -> Optional[str]:
         )
 
         return release_notes_builder.build()
+
+    def get_latest_release(self, repo: Repository) -> Optional[GitRelease]:
+        """
+        Get the latest release of the repository.
+
+        @param repo: The repository to get the latest release from.
+        @return: The latest release of the repository, or None if no releases are found.
+        """
+        # check if from-tag name is defined
+        if ActionInputs.is_from_tag_name_defined():
+            logger.info("Getting latest release by from-tag name %s", ActionInputs.get_tag_name())
+            rls: GitRelease = self._safe_call(repo.get_release)(ActionInputs.get_from_tag_name())
+
+            if rls is None:
+                logger.info("Latest release not found for received tag %s. Ending!", ActionInputs.get_from_tag_name())
+                sys.exit(1)
+
+        else:
+            logger.info("Getting latest release by semantic ordering (could not be the last one by time).")
+            gh_releases: list = list(self._safe_call(repo.get_releases)())
+            rls: GitRelease = self.__get_latest_semantic_release(gh_releases)
+
+            if rls is None:
+                logger.info("Latest release not found for %s. 1st release for repository!", repo.full_name)
+
+        if rls is not None:
+            logger.debug(
+                "Latest release with tag:'%s' created_at: %s, published_at: %s",
+                rls.tag_name,
+                rls.created_at,
+                rls.published_at,
+            )
+
+        return rls
+
+    def __get_latest_semantic_release(self, releases) -> Optional[GitRelease]:
+        published_releases = [release for release in releases if not release.draft and not release.prerelease]
+        latest_version: Optional[semver.Version] = None
+        rls: Optional[GitRelease] = None
+
+        for release in published_releases:
+            try:
+                version_str = release.tag_name.lstrip("v")
+                current_version: Optional[semver.Version] = semver.VersionInfo.parse(version_str)
+            except ValueError:
+                logger.debug("Skipping invalid value of version tag: %s", release.tag_name)
+                continue
+            except TypeError:
+                logger.debug("Skipping invalid type of version tag: %s", release.tag_name)
+                continue
+
+            if latest_version is None or current_version > latest_version:
+                latest_version = current_version
+                rls = release
+
+        return rls

release_notes_generator/record/record_factory.py

@@ -68,7 +68,7 @@ def create_record_for_issue(r: Repository, i: Issue) -> None:
             @return: None
             """
             # check for skip labels presence and skip when detected
-            issue_labels = [label.name for label in issue.labels]
+            issue_labels = [label.name for label in i.labels]
             skip_record = any(item in issue_labels for item in ActionInputs.get_skip_release_notes_labels())
             records[i.number] = Record(r, i, skip=skip_record)
 

release_notes_generator/utils/constants.py

@@ -22,6 +22,7 @@
 GITHUB_REPOSITORY = "GITHUB_REPOSITORY"
 GITHUB_TOKEN = "github-token"
 TAG_NAME = "tag-name"
+FROM_TAG_NAME = "from-tag-name"
 CHAPTERS = "chapters"
 DUPLICITY_SCOPE = "duplicity-scope"
 DUPLICITY_ICON = "duplicity-icon"

requirements.txt

@@ -6,3 +6,4 @@ pylint==3.2.6
 requests==2.31.0
 black==24.8.0
 PyYAML==6.0.2
+semver==3.0.2