Skip to content

ci: bring over some workflows from AIT#41

Merged
duzos merged 1 commit intomasterfrom
ci/workflows
Jul 6, 2025
Merged

ci: bring over some workflows from AIT#41
duzos merged 1 commit intomasterfrom
ci/workflows

Conversation

@duzos
Copy link
Member

@duzos duzos commented Jul 6, 2025

No description provided.

Copilot AI review requested due to automatic review settings July 6, 2025 15:34
@duzos duzos merged commit 187e175 into master Jul 6, 2025
@duzos duzos deleted the ci/workflows branch July 6, 2025 15:36
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR brings over and updates various CI workflows from the AIT project, and adds a Python script to automate changelog entries.

  • Introduces scripts/update_changelog.py and its requests dependency
  • Adds/updates multiple GitHub Actions workflows for bug form updates, changelog automation, labeling, and publishing
  • Removes several deprecated workflows and updates templates and funding configuration

Reviewed Changes

Copilot reviewed 17 out of 18 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
scripts/update_changelog.py New Python script to append PR entries to CHANGELOG
scripts/requirements.txt Adds requests~=2.32.3 dependency
.github/workflows/update-bug-form.yml Workflow to append mod version to bug report form
.github/workflows/publish_release_github.yml Removed old “Publish Release on GitHub” workflow
.github/workflows/publish_beta_github.yml Removed old “Publish Beta on GitHub” workflow
.github/workflows/publish_beta_discord.yml Removed old Discord beta-build workflow
.github/workflows/publish.yml Consolidated publish workflow for GitHub/Modrinth/CurseForge
.github/workflows/publish-devbuilds.yml Updated dev-builds workflow to “donator builds”
.github/workflows/pr-labeler.yml Adds PR labeler workflow
.github/workflows/clear_changelog.yml Workflow to clear CHANGELOG after release merges
.github/workflows/append_changelog.yml Workflow to append merged PRs to CHANGELOG.md
.github/workflows/all-labeler-untriaged.yml Workflow to auto-label untriaged issues/PRs
.github/labeler.yml File-change-based PR label mappings
.github/PULL_REQUEST_TEMPLATE.md Updated pull request template
.github/ISSUE_TEMPLATE/2-feature-request.yaml New feature-request issue template
.github/ISSUE_TEMPLATE/1-bug-report.yaml New bug-report issue template
.github/FUNDING.yml Updates Patreon and Ko-fi entries
Comments suppressed due to low confidence (6)

.github/workflows/update-bug-form.yml:29

  • The echoed filename 1_bug_report.yaml does not match the actual filename 1-bug-report.yaml; update the message for accuracy.
            echo "Version $MOD_VERSION already exists in 1_bug_report.yaml"

.github/workflows/all-labeler-untriaged.yml:14

  • GitHub Actions expressions do not support a join function; this condition will fail. Use a supported expression to test for absent labels.
        if: join(github.event.issue.labels) == ''

.github/ISSUE_TEMPLATE/1-bug-report.yaml:41

  • The default value 0 isn’t one of the listed version options; this may confuse users. Set a valid default or remove it.
    default: 0

.github/ISSUE_TEMPLATE/2-feature-request.yaml:27

  • The checkboxes input is missing an id field, which is required for form processing. Add an id: to this block.
- type: checkboxes

.github/workflows/publish-devbuilds.yml:63

  • [nitpick] There is a large commented-out alternative Gradle setup block that may clutter the workflow. Consider removing or moving to documentation.
    # NOTE: The Gradle Wrapper is the default and recommended way to run Gradle (https://docs.gradle.org/current/userguide/gradle_wrapper.html).

.github/PULL_REQUEST_TEMPLATE.md:16

  • [nitpick] The guidelines link points to amblelabs.github.io/ait-wiki; verify that this is the correct and accessible documentation location for this repo.
- [ ] I have read and am following the [Pull Request and Changelog Guidelines](https://amblelabs.github.io/ait-wiki/guidelines).

Comment on lines +67 to +68
suffix = f" by {authors_md}" if authors_md else ""
lines.append(f"- {clean} |{suffix} ([#{num}]({link}))")
Copy link

Copilot AI Jul 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[nitpick] The formatting always adds a | even when there are no authors; consider only including the pipe and suffix when authors_md is non-empty to avoid stray separators.

Suggested change
suffix = f" by {authors_md}" if authors_md else ""
lines.append(f"- {clean} |{suffix} ([#{num}]({link}))")
suffix = f" | by {authors_md}" if authors_md else ""
lines.append(f"- {clean}{suffix} ([#{num}]({link}))")

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants