Awesome/awesome-copilot
1
0
Fork 0
mirror of https://github.com/github/awesome-copilot.git synced 2026-03-14 05:05:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor: extract Learning Hub article ordering into shared config

Browse Source 
Move duplicated fundamentalsOrder and referenceOrder arrays from
index.astro and ArticleLayout.astro into a shared config file at
src/config/learning-hub.ts. Both consumers now import from the
single source of truth.

Addresses PR review comment about maintenance burden of keeping
two copies in sync.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This commit is contained in:
Aaron Powell
2026-02-26 14:59:50 +11:00
parent e0636e8165
commit 0637cad186
3 changed files with 22 additions and 36 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
website/src/config/learning-hub.ts Normal file
View File

@@ -0,0 +1,20 @@
// Shared article ordering for the Learning Hub.
// Used by both the index page and the article sidebar layout.
export const fundamentalsOrder = [
'what-are-agents-skills-instructions',
'understanding-copilot-context',
'copilot-configuration-basics',
'defining-custom-instructions',
'creating-effective-skills',
'building-custom-agents',
'understanding-mcp-servers',
'automating-with-hooks',
'using-copilot-coding-agent',
'installing-and-using-plugins',
'before-after-customization-examples',
];
export const referenceOrder = [
'github-copilot-terminology-glossary',
];

19
website/src/layouts/ArticleLayout.astro
View File

@@ -1,6 +1,7 @@
--- ---
import BaseLayout from './BaseLayout.astro'; import BaseLayout from './BaseLayout.astro';
import { getCollection } from 'astro:content'; import { getCollection } from 'astro:content';
import { fundamentalsOrder, referenceOrder } from '../config/learning-hub';
interface Props { interface Props {
title: string; title: string;
@@ -15,24 +16,6 @@ const base = import.meta.env.BASE_URL;
const articles = await getCollection('learning-hub'); const articles = await getCollection('learning-hub');
const fundamentalsOrder = [
'what-are-agents-skills-instructions',
'understanding-copilot-context',
'copilot-configuration-basics',
'defining-custom-instructions',
'creating-effective-skills',
'building-custom-agents',
'understanding-mcp-servers',
'automating-with-hooks',
'using-copilot-coding-agent',
'installing-and-using-plugins',
'before-after-customization-examples',
];
const referenceOrder = [
'github-copilot-terminology-glossary',
];
const fundamentals = articles const fundamentals = articles
.filter((a) => fundamentalsOrder.includes(a.id)) .filter((a) => fundamentalsOrder.includes(a.id))
.sort((a, b) => fundamentalsOrder.indexOf(a.id) - fundamentalsOrder.indexOf(b.id)); .sort((a, b) => fundamentalsOrder.indexOf(a.id) - fundamentalsOrder.indexOf(b.id));

19
website/src/pages/learning-hub/index.astro
View File

@@ -1,28 +1,11 @@
--- ---
import BaseLayout from '../../layouts/BaseLayout.astro'; import BaseLayout from '../../layouts/BaseLayout.astro';
import { getCollection } from 'astro:content'; import { getCollection } from 'astro:content';
import { fundamentalsOrder, referenceOrder } from '../../config/learning-hub';
const base = import.meta.env.BASE_URL; const base = import.meta.env.BASE_URL;
const articles = await getCollection('learning-hub'); const articles = await getCollection('learning-hub');
const fundamentalsOrder = [
'what-are-agents-skills-instructions',
'understanding-copilot-context',
'copilot-configuration-basics',
'defining-custom-instructions',
'creating-effective-skills',
'building-custom-agents',
'understanding-mcp-servers',
'automating-with-hooks',
'using-copilot-coding-agent',
'installing-and-using-plugins',
'before-after-customization-examples',
];
const referenceOrder = [
'github-copilot-terminology-glossary',
];
const fundamentals = articles const fundamentals = articles
.filter((a) => fundamentalsOrder.includes(a.id)) .filter((a) => fundamentalsOrder.includes(a.id))
.sort((a, b) => fundamentalsOrder.indexOf(a.id) - fundamentalsOrder.indexOf(b.id)); .sort((a, b) => fundamentalsOrder.indexOf(a.id) - fundamentalsOrder.indexOf(b.id));