Skip to content

Sort community skills alphabetically - #122

Open
drQedwards wants to merge 1 commit into
stellar:mainfrom
drQedwards:sort-ecosystem-cards-alphabetically
Open

Sort community skills alphabetically#122
drQedwards wants to merge 1 commit into
stellar:mainfrom
drQedwards:sort-ecosystem-cards-alphabetically

Conversation

@drQedwards

Copy link
Copy Markdown
Contributor

Why this PR

Follow-up to #111 implementing @kaankacar's request in #111 (comment).

#111 conflicts with main after #103 (copyValue blob URLs → raw.githubusercontent.com). Kan asked to rebase on main and apply the sort there, not resolve the conflict line by line (that would restore blob URLs). I cannot push to @mariaelisaaraya's fork, so this is the same change on current main.

What

No CI drift check here — Kan said that can be its own PR.

Test plan

  • 29 cards, titles sorted case-insensitive
  • no github.com/.../blob/ in ECOSYSTEM_CARDS
  • pnpm check:ecosystem-links / pnpm lint / pnpm lint:ts / pnpm build on site-ci after this push

Rebase of stellar#111 onto main after stellar#103: keep raw.githubusercontent.com
copyValue URLs, insert community cards by title, and stop README/CLAUDE
from telling contributors to append at the end.
Copilot AI balanced review requested due to automatic review settings August 31, 2026 23:52
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@kaankacar

Copy link
Copy Markdown
Contributor

🤖 Automated message from Kaan's Automated Triage Bot.

👀 Picked this up — a review will follow shortly.

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@kaankacar

Copy link
Copy Markdown
Contributor

🤖 Automated message from Kaan's Automated Triage Bot.

Thanks for doing this rebase. It is the clean version that #111 was asked
for, and the sort itself verifies.

What I checked at 3924c13:

  • One commit on current main, no conflict.
  • Pure permutation: the same 29 cards, every field byte-identical.
  • Case-insensitive title order is correct at all 28 adjacent pairs.
  • SKILL_CARD_SOURCES untouched, zero blob URLs.
  • StellarTools loses exactly two invisible characters (U+202F, U+2060).
    No visible text changes.
  • site-ci is green at this head: lint, lint:ts, check:ecosystem-links,
    test:ecosystem-links and build all ran.

I am not merging, because two calls are not mine to make:

One correction on the description: the comment you cite is from this triage
bot, not from @kaankacar. It did ask #111's author to rebase, and it ends by
keeping the hold. So the rebase request is real, but no merge decision was
made.

One fact that helps that decision: no open PR touches ECOSYSTEM_CARDS
today. #116, #55 and #49 only touch FILTERS and SKILL_CARD_SOURCES. So
the usual conflict cost of a whole-region reorder is near zero right now.

@kaankacar over to you.

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.

3 participants