Peter Newman

Results 814 comments of Peter Newman

@mathertel this needs re-opening as #15 doesn't actually fix it unfortunately.

Bump @mathertel , can you reopen this please?

There's a flake8 action with an annotation here: https://github.com/TrueBrain/actions-flake8/blob/main/flake8-matcher.json To make it work: https://github.com/TrueBrain/actions-flake8/blob/7e33a2bcf3bee056945183e6d08d70000de48114/entrypoint.sh#L3-L6 https://github.com/TrueBrain/actions-flake8/blob/7e33a2bcf3bee056945183e6d08d70000de48114/entrypoint.sh#L32-L34

Also see these matchers: https://github.com/PyCQA/pylint/pull/2758#issuecomment-696059847

> PRs in the profile are shown in chronological order before a user wins, not the order in which they will mature. Agreed, I'm suggesting as a user I'm interested...

> Ah, I misunderstood then, you want all PRs on the profile to be sorted by when they will mature? Yeah, well my main thing is I think the four...

> I would also cancel the timer for merged PRs. What does it protect against, malicious co-maintainers? Right now there's a large amount of uncertainty -- you might have seven...

> but it also gives us more time and flexibility to react to any further spam issues that we encounter and to block and repositories that we believe don't follow...

It looks like that line should probably actually be: ` memset(&tod.data.toddata.tod,0x00, ARTNET_MAX_UID_COUNT * ARTNET_RDM_UID_WIDTH);` If someone wants to let us know if that works or better yet open a pull...

> Egads, I can't figure out how to make an "unresolved conversation." Sorry about the noise. I think you're talking about a line comment: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/commenting-on-a-pull-request#adding-line-comments-to-a-pull-request