XSF Discussion - 2020-12-16

anyone wants to review and merge my PR here? https://github.com/xsf/xmpp.org/pull/853

ME! ME! PICK ME!

oh

Mouhahahaha

Fear me, I ruin all the fun!

But, yeah, looks good to me. Is that Unix/Linux thing a result of the linting?

Ah, yes.

I'm constantly lagging behind the Github changes, it seems 🙂

ya, linter fails and complains that UNIX is not defined in platforms

Whaaaa

Didn't council change at all?

Nope

not sure where I can find the failing travis validations

I'm happy to merge as-is, unless someone thinks it is important to pull out the platform change?

I can't see why - but I'll hold off pressing the big green button for a bit...

I only fixed it because I fell bad when PRs with failing CI are getting merged ;-)

here you can see an old CI build: https://travis-ci.org/github/xsf/xmpp.org/jobs/749881434

``` $ if [[ $TRAVIS_PYTHON_VERSION == 3.7 ]]; then ./data/lint-list.py clients.json; fi ERROR: entry 'txmmp': undefined platforms: 'Unix' (the allowed platforms are listed in platforms.json. If you think a platform is missing add it and mention it in your Pull Request) Found 1 severe violations. Please fix them. ```

we can also add Unix to the platform.json

let’s not lose valuable contributor time in such minor discussions

especially since it’s neatly separated into separate commits

merged

thanks

ping me in a few so that I remember to pull the website

thanks

Thank you

Will update the Board section on this site hopefully tomorrow: https://xmpp.org/about/xmpp-standards-foundation.html

Why didn't it complain about adding "Unix" then???

maybe it did and people ignored the CI?

I think so, I can merge with CI even failing. We should consider making CI not fail a requirement

sounds like a reasonable plan

Meh. CI is a guide, not an authority

Guus, I tend to disagree :)

especially those checks

I don’t have the necessary privileges to do that

this is why you can have warnings and errors in CI

CI OK'd https://github.com/xsf/xmpp.org/pull/849/commits/ffcd26776137877f0f8c2890093f372ce9a7a67f

"nice"

(and is kinda surprised it wasn't him causing the issue)

Hey, come back and accept responsibility!!!11!!eleventy!

Zash, on the last commit is sais "all checks failed"

✏

updated

why is txmpp even there?

Has the build ever far even to go look like done?

I just waited a few.

Ge0rG I think there was debate in a github issue around that. I recall that not being a hill that I was going to die on, if memory serves.

Guus: I remember that debate, but it lacked a conclusion

so somebody pushed "merge" I suppose.

likely me?

do you guys are aware of some XMPP clients that are requestion vcards on the service level ?

*requesting

for mucs?