Skip to content

Conversation

@PascalinDe
Copy link
Member

adds/updates:

  • showcase v2.0
  • matrix.epfl.ch
  • eID innosuisse
  • Orchard
  • student project spring 2025
  • e-voting
  • anyway

closes https://github.com/c4dt/TODO/issues/527
closes c4dt/showcase_v2#222

@PascalinDe PascalinDe requested a review from ineiti October 17, 2025 08:36
@github-project-automation github-project-automation bot moved this to Active Tasks in Engineer's TODO Oct 17, 2025
@PascalinDe PascalinDe moved this from Active Tasks to Ready4Review in Engineer's TODO Oct 17, 2025
Copy link
Member

@ineiti ineiti left a comment

Choose a reason for hiding this comment

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

Generic comment:

  • Please use a hard wrap of 80 or 72 characters
  • Every sentence should start on a new line

This helps reading on badly configured editors and makes commits more concise and readable.

I cannot get prettier to line-wrap on new sentences. So this is the next best I can come up with:

prettier --prose-wrap always --print-width 80 --parser html anyway.tpl

The eid-Innosuisse entry is awful - I'll have to fix that...

@PascalinDe
Copy link
Member Author

updated the template files as you requested, hope I understood your suggestion regarding the sentences correctly

I didn't change the projects.yaml files, as they already had a limit on 130, that I respected

Copy link
Member

@ineiti ineiti left a comment

Choose a reason for hiding this comment

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

Great - looks much better now, just in case we'd want to edit the files later.

Did you do the sentence break manually? I didn't find the flag for prettier to do so.

@PascalinDe PascalinDe merged commit 7afb741 into main Oct 22, 2025
1 check passed
@PascalinDe PascalinDe deleted the 222 branch October 22, 2025 16:34
@github-project-automation github-project-automation bot moved this from Ready4Review to Done in Engineer's TODO Oct 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Add texts to old showcase

3 participants