Skip to content

ci(workflow): add common workflows for Android and Python, including CI /CD revamp#422

Merged
wax911 merged 6 commits into
developfrom
ci/consolidate-workflows
Sep 27, 2025
Merged

ci(workflow): add common workflows for Android and Python, including CI /CD revamp#422
wax911 merged 6 commits into
developfrom
ci/consolidate-workflows

Conversation

@wax911

@wax911 wax911 commented Sep 27, 2025

Copy link
Copy Markdown
Member

AniTrend Pull Request

Thank you for contributing! Please take a moment to review our contributing guidelines to make the process easy and effective for everyone involved.

Please open an issue before embarking on any significant pull request, especially those that add a new library or change existing tests, otherwise you risk spending a lot of time working on something that might not end up being merged into the project.

Before opening a pull request, please ensure you've done the following:

  • You have followed our contributing guidelines
  • Double checked that your branch is based on develop and targets develop (where applicable)
  • Pull request has tests (if applicable)
  • Documentation is updated (if necessary)
  • Description explains the issue/use-case resolved

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Enhancement (Improves existing functionality)

IMPORTANT: By submitting a patch, you agree to allow the project owners to license your work under the terms of the Apache License 2.0.

Copilot AI review requested due to automatic review settings September 27, 2025 12:06
@anitrend anitrend Bot added the docs label Sep 27, 2025

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request consolidates and modernizes the CI/CD workflows for both Android and Python components of the project. It introduces reusable GitHub Actions, streamlines workflow organization, and establishes consistent patterns across the repository.

  • Consolidated multiple individual Android workflows into a single comprehensive android-ci.yml workflow
  • Replaced the existing Python workflow with new python-ci.yml and updated python-cd.yml workflows
  • Added reusable composite actions for common setup tasks and centralized configuration files

Reviewed Changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
AGENTS.md Added comprehensive repository guidelines covering project structure, build commands, coding standards, and development workflows
.java-version Added Java 21 version specification for consistent tooling
.github/workflows/python-ci.yml New comprehensive Python CI workflow with dependency review, CodeQL analysis, linting, and testing
.github/workflows/python-cd.yml Updated Python CD workflow to use the new reusable action and simplified token handling
.github/workflows/greeting.yml Minor updates to workflow name and job naming
.github/workflows/gradle-wrapper-validation.yml Removed standalone wrapper validation (now part of android-ci)
.github/workflows/gradle-dokka.yml Removed standalone documentation generation (now part of android-ci)
.github/workflows/emoji-generator-ci.yml Removed old Python workflow (replaced by python-ci)
.github/workflows/android-unit-test.yml Removed standalone unit test workflow (now part of android-ci)
.github/workflows/android-spotless.yml Removed standalone spotless workflow (now part of android-ci)
.github/workflows/android-publish-artifact.yml Removed standalone artifact publishing (now part of android-ci)
.github/workflows/android-ci.yml New consolidated Android CI workflow combining all previous Android workflows
.github/actions/python/action.yml New reusable composite action for Python environment setup
.github/actions/android/action.yml New reusable composite action for Android environment setup

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread .github/actions/python/action.yml Outdated
Comment thread .github/actions/android/action.yml Outdated
Comment thread .github/actions/python/action.yml Outdated
Comment thread .github/workflows/python-ci.yml
@github-advanced-security

Copy link
Copy Markdown

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@wax911 wax911 force-pushed the ci/consolidate-workflows branch 6 times, most recently from 2dd0a30 to c56f1c7 Compare September 27, 2025 13:14
@wax911 wax911 force-pushed the ci/consolidate-workflows branch from c56f1c7 to 2346d7c Compare September 27, 2025 13:22

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Auto approved automated PR

@wax911 wax911 merged commit 9b9da78 into develop Sep 27, 2025
17 of 19 checks passed
@wax911 wax911 deleted the ci/consolidate-workflows branch September 27, 2025 14:13
wax911 added a commit that referenced this pull request Sep 27, 2025
…CI /CD revamp (#422)

* ci(workflow): add common workflows for Android and Python, including CI /CD revamp

* ci(workflow): ignore python CI workflow in push and pull request triggers

* ci(actions): add caching for Gradle dependencies in Android workflow

* ci(workflow): update python CI workflow to include additional paths and job names

* ci(workflow): standardize workflow names to title case

* ci(workflow): seperate lint job, format and type checks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants