search

sagemath / website

Source for https://www.sagemath.org/; submit Issues/PRs here to update the website
40 stars 180 forks source link

Merge changelogs list #350

Closed kwankyu closed 1 year ago

kwankyu commented 1 year ago

I mage the merge with the changelogs, omitting some wrong entries.

Then I removed a few obvious duplicates, but I guess there are many new duplicates.

At the end of the list, there are some names which look like trac account names. They may be removed. But I did not.

kwankyu commented 1 year ago

At the end of the list, there are some names which look like trac account names. They may be removed. But I did not.

What should we do about these names?

mkoeppe commented 1 year ago

What should we do about these names?

I think they should become <contributor trac="..."/>

haraldschilly commented 1 year ago

@kwankyu I merged this, but then I discovered xml is broken. e.g. github="ypfmde"/>; ends with a semicolon, that's not ok. I'll fix it.

kwankyu commented 1 year ago

@kwankyu I merged this, but then I discovered xml is broken. e.g. github="ypfmde"/>; ends with a semicolon, that's not ok. I'll fix it.

Right. Thanks!

kwankyu commented 1 year ago

What should we do about these names?

I think they should become <contributor trac="..."/>

Okay. I will do that, and do the fix in the comment

https://github.com/sagemath/website/pull/346#issuecomment-1280443514

in a new PR.