Skip to content

Fix present #725

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Fix present #725

wants to merge 10 commits into from

Conversation

CxRes
Copy link
Member

@CxRes CxRes commented Apr 6, 2025

Fixes present entries for a lot of participants that I was reasonably certain of.

@mrkvon @thhck You might want to check and PR yours.

@TallTed Your entries were especially difficult. While I have cleaned it to two types, I am not sure which one you prefer.

@thhck
Copy link

thhck commented Apr 7, 2025

I would not be in favor of removing the double space at the end of the lines, since this is the way in traditional markdown to indicate newline.

Maybe github flavored markdown don't needs the double space, but if we want to reuse those markdown files outside of github it would be nice to keep the double spaces

@TallTed
Copy link
Contributor

TallTed commented Apr 7, 2025

I would not be in favor of removing the double space at the end of the lines, since this is the way in traditional markdown to indicate newline.

I would suggest changing the invisible double-space-EOL to visible <br>EOL.

Copy link
Member

@csarven csarven left a comment

Choose a reason for hiding this comment

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

There is the Solid Ecosystem Monitor which lists both scribes and participants.

The names of the participants in the minutes are meant to match with the names on their W3C account - easy check for Solid CG participants. Links to names are optional but encouraged. Other info is fine too.

Since the ecosystem monitor processes all the minutes, it helps a lot to easily spot errors in the minutes. The tool can be improved of course, so that's welcome needless to say.

@CxRes
Copy link
Member Author

CxRes commented Apr 7, 2025

@csarven Most of the fixes that I have made were done with the help of the Solid Ecosystem Monitor. Already this took quite a lot of time.

@CxRes
Copy link
Member Author

CxRes commented Apr 7, 2025

@thhck Thanks for pointing out double spaces being removed, my code editor is set up to automatically remove spaces at the end of lines. But I agree with @TallTed that adding <br> is better.

UPDATE: found only two instances of Double Space EOLs which were fixed.

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.

5 participants