Skip to content

Conversation

@igorpecovnik
Copy link
Member

@igorpecovnik igorpecovnik commented Feb 1, 2026

Summary

  • Update community minimal CLI target from Debian Forky to Debian Trixie
  • Remove Forky fallback patterns from exposed.map generation
  • Rename community-forky-allcommunity-trixie-all

Trixie is now the primary testing release, replacing Forky as the fallback.

Test plan

  • Verify community-trixie-all target generates correctly
  • Check exposed.map patterns use trixie instead of forky

Update the community minimal CLI target to use Debian Trixie instead
of Debian Forky. Also removes Forky fallback patterns from exposed.map
generation since Trixie is now the primary testing release.

- community-forky-all → community-trixie-all
- RELEASE: forky → RELEASE: trixie
- Remove forky fallback patterns (trixie is primary)

Signed-off-by: Igor Pecovnik <igor@armbian.com>
@github-actions github-actions bot added 02 Milestone: First quarter release size/small PR with less then 50 lines Needs review Seeking for review labels Feb 1, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 1, 2026

Walkthrough

The script scripts/generate_targets.py is updated to upgrade the Debian community target from Forky to Trixie. The community target name is changed from community-forky-all to community-trixie-all, the RELEASE variable is updated from forky to trixie, and related descriptive comments and section headers are adjusted. Additionally, fallback minimal patterns for the forky variant are removed from the exposed map generation, eliminating Armbian board-specific forky minimal patterns and relying solely on trixie-based patterns.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title 'targets: change community builds from forky to trixie' directly and accurately summarizes the main change: updating community builds from Debian Forky to Debian Trixie.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Description check ✅ Passed The pull request description clearly relates to the changeset, describing the update from Forky to Trixie targets and removal of Forky fallback patterns.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch commm

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@igorpecovnik igorpecovnik merged commit 5523fb8 into main Feb 1, 2026
6 of 7 checks passed
@igorpecovnik igorpecovnik deleted the commm branch February 1, 2026 11:04
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@scripts/generate_targets.py`:
- Around line 1059-1072: Update the documentation references from the old target
name to the new one: replace occurrences of community-forky-all with
community-trixie-all in release-targets/README.md (specifically in the
"Community-maintained builds" section) and ensure any explanatory text or
examples that mention the target name are updated to reflect the new vars
(RELEASE: trixie, BUILD_MINIMAL, BUILD_DESKTOP) used by the community-trixie-all
entry in scripts/generate_targets.py.

Comment on lines +1059 to +1072
# Debian trixie minimal CLI for all community boards
community-trixie-all:
enabled: yes
configs: [ armbian-community ]
pipeline:
gha: *armbian-gha
build-image: "yes"
vars:
RELEASE: forky
RELEASE: trixie
BUILD_MINIMAL: "yes"
BUILD_DESKTOP: "no"
items:
"""
# Combine all lists for Debian Forky
# Combine all lists for Debian Trixie
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Search for references to the old target name "community-forky-all"
echo "=== Searching for 'community-forky-all' in entire repository ==="
rg 'community-forky-all' -i

echo ""
echo "=== Searching for 'community-forky' specifically ==="
rg 'community-forky' -i

echo ""
echo "=== Checking .github directory for workflow references ==="
fd . '.github' -type f | head -20

Repository: armbian/armbian.github.io

Length of output: 617


🏁 Script executed:

#!/bin/bash
cat -n release-targets/README.md | head -50
echo ""
echo "=== Searching for context around community builds ==="
rg -A 2 -B 2 'community-' release-targets/README.md

Repository: armbian/armbian.github.io

Length of output: 2924


Documentation update required before merge.

The code changes are internally consistent, but release-targets/README.md still references the old target name community-forky-all. Update the documentation to reflect the new target name community-trixie-all:

Required documentation update

In release-targets/README.md, update the Community-maintained builds section:

- `community-forky-all`: Debian Forky minimal CLI for all boards
+ `community-trixie-all`: Debian Trixie minimal CLI for all boards
🤖 Prompt for AI Agents
In `@scripts/generate_targets.py` around lines 1059 - 1072, Update the
documentation references from the old target name to the new one: replace
occurrences of community-forky-all with community-trixie-all in
release-targets/README.md (specifically in the "Community-maintained builds"
section) and ensure any explanatory text or examples that mention the target
name are updated to reflect the new vars (RELEASE: trixie, BUILD_MINIMAL,
BUILD_DESKTOP) used by the community-trixie-all entry in
scripts/generate_targets.py.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

02 Milestone: First quarter release Needs review Seeking for review size/small PR with less then 50 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants