FSFE20 pages #1917

Merged
max.mehl merged 13 commits from nico.rikken/fsfe-website:feature/fsfe20-recovery into master 2 years ago
Collaborator

Commit of generic FSFE20 pages, which also aggregate related articles. This pull request replaces the earlier pull request #1905

Commit of generic FSFE20 pages, which also aggregate related articles. This pull request replaces the earlier pull request https://git.fsfe.org/FSFE/fsfe-website/pulls/1905
nico.rikken added 12 commits 2 years ago
max.mehl approved these changes 2 years ago
max.mehl left a comment
Owner

LGTM, thank you!

One question from my side, but I leave it to you to decide this.

LGTM, thank you! One question from my side, but I leave it to you to decide this.
news/2021/*:[fsfe20]
Owner

Why do you limit it to these two years? Usually, we wildcard this, shouldn't be a huge performance loss AFAIK.

Why do you limit it to these two years? Usually, we wildcard this, shouldn't be a huge performance loss AFAIK.
Collaborator

FWIW, for news it is a measurable performance loss, because it means that very many XML files have to be parsed.

FWIW, for news it *is* a measurable performance loss, because it means that very many XML files have to be parsed.
Owner

Ah, good to know! I thought this wasn't intensive.

Silly side question: is it possible to give it a range, like 2019-*? In this case, we could surely identify some tags that only start to exist after the first 15 years of news items and events ;)

Ah, good to know! I thought this wasn't intensive. Silly side question: is it possible to give it a range, like `2019-*`? In this case, we could surely identify some tags that only start to exist after the first 15 years of news items and events ;)
Collaborator

No, there's only shell globbing, so what you could do maybe was have 2 lines with 2019 and 202*, which will work for the upcoming 8.5 years.

No, there's only shell globbing, so what you could do maybe was have 2 lines with `2019` and `202*`, which will work for the upcoming 8.5 years.
Owner

That sounds reasonable!

@nico.rikken would it be possible for you to implement that? This would avoid the risk that we may have an article tagged with this in 2022, which could well happen in a restrospective or so.

That sounds reasonable! @nico.rikken would it be possible for you to implement that? This would avoid the risk that we may have an article tagged with this in 2022, which could well happen in a restrospective or so.
Poster
Collaborator

Sure I will. I found another .sources file somewhere also having a limited date scope. Nothing in 2019 or 2020 was tagged with fsfe20, so 202* will do just fine. I just assumed nothing would be written after 2022. I'll convert it to a globbing line then.

Sure I will. I found another `.sources` file somewhere also having a limited date scope. Nothing in 2019 or 2020 was tagged with `fsfe20`, so `202*` will do just fine. I just assumed nothing would be written after 2022. I'll convert it to a globbing line then.
nico.rikken marked this conversation as resolved
nico.rikken added 1 commit 2 years ago
continuous-integration/drone/pr Build is passing Details
2f5c0db304
chore: simplify sources file using globbing
max.mehl merged commit ee070018ae into master 2 years ago
max.mehl referenced this issue from a commit 2 years ago

Reviewers

max.mehl approved these changes 2 years ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as ee070018ae.
Sign in to join this conversation.
Loading…
There is no content yet.