19,158 matching drops
#1459158 2026-09-22 15:17
punk6529 Dev Team Chat #1124898 2026-06-23 08:05
@[prxt0] / all, take a look and try, I think this is the right way to do it
punk6529 Dev Team Chat #1124896 2026-06-23 08:05
Set up in GitHub. Created team `6529seize Maintainers` / `6529seize-maintainers` with repo permission `maintain` on `6529-Collections/6529seize-frontend`. Members active: `punk6529`, `ragnep`, `GelatoGenesis`, `simo6529`, `prxt6529` Effective repo roles verified: `punk6529`: `admin`, unchanged `GelatoGenesis`: `admin`, unchanged `ragnep`: `maintain` `simo6529`: `maintain` `prxt6529`: `maintain` Also replaced the old `main` branch protection with an active repository ruleset: `main maintainer review and merge policy` Ruleset ID: `18018081` It applies to the default branch and enforces: - PR required before merging - 1 approval required - approval must come from `6529seize Maintainers` - stale approvals dismissed on push - last pusher approval protection - review threads must be resolved - required checks preserved: `DCO`, `security/snyk (6529)` - no direct updates to `main` except via PR-only bypass - no deletes / force pushes The maintainer team has **PR-only bypass**, matching GitHub’s intended ruleset model: they can override and merge through a PR, while non-maintainers need one of them involved.
punk6529 Dev Team Chat #1122284 2026-06-22 15:08
i think bot is amusing itself with E2E stuff
punk6529 Dev Team Chat #1122275 2026-06-22 15:07
no, I don't think so
punk6529 Dev Team Chat #1122260 2026-06-22 15:04
it can be in a separate PR
punk6529 Dev Team Chat #1122257 2026-06-22 15:04
why not ask codex to fix the unrelated failures?
punk6529 Dev Team Chat #1122243 2026-06-22 14:59
in a way that was simply not possible before
punk6529 Dev Team Chat #1122242 2026-06-22 14:59
is that we can do mass scale improvements
punk6529 Dev Team Chat #1122241 2026-06-22 14:59
and the general message I am here to preach
punk6529 Dev Team Chat #1122240 2026-06-22 14:59
if the answer is a), then we need to have a different chat :slightly_smiling_face:
punk6529 Dev Team Chat #1122239 2026-06-22 14:58
I am almost certain the answer is b)
punk6529 Dev Team Chat Reply #1122238 2026-06-22 14:58
what are the possible reasons that this was not done by you guys: a) you do not believe that testing is an important part of software development b) you thought it was going to be quite a bit of work and you were pressured by other priorities
Context from punk6529 #1122224 2026-06-22 14:56
1/ last 60 hours completely rebuild the testing framework, a process that is I dunno 60-70% done 2/ @[prxt0] raises a PR during that process which fails a CI gate 3/ the PR is not, as far as I can tell, particularly urgent or important 4/ among the various available options: a) do not merge it yet b) fix the failing tests c) break my fairly complex in-progress rebuild of the testing framework, @[prxt0] suggest 4c
punk6529 Dev Team Chat Reply #1122233 2026-06-22 14:58
i have no idea; have not looked at codex since this morning
Context from prxt0 #1122232 2026-06-22 14:57
can i just check something? are tests supposed to be all fixed now? i.e. if i run full test suite on main i should expect no failures?
punk6529 Dev Team Chat #1122231 2026-06-22 14:57
the point that the whole team @[simo] @[GelatoGenesis] @[prxt0] should all take away as a learning was "why did you not harden and deepen and industrialize the testing framework before I came in and did it"
punk6529 Dev Team Chat #1122227 2026-06-22 14:56
the point I am trying to make has nothing to do with this morning's PR
punk6529 Dev Team Chat #1122225 2026-06-22 14:56
anyway, I am not actually upset at you to be clear
punk6529 Dev Team Chat #1122224 2026-06-22 14:56
1/ last 60 hours completely rebuild the testing framework, a process that is I dunno 60-70% done 2/ @[prxt0] raises a PR during that process which fails a CI gate 3/ the PR is not, as far as I can tell, particularly urgent or important 4/ among the various available options: a) do not merge it yet b) fix the failing tests c) break my fairly complex in-progress rebuild of the testing framework, @[prxt0] suggest 4c
punk6529 Dev Team Chat #1122221 2026-06-22 14:54
the correct series of events is the following
punk6529 Dev Team Chat #1122220 2026-06-22 14:54
did I tell you to raise a PR before I finished doing the testing framework?
punk6529 Dev Team Chat Reply #1122218 2026-06-22 14:53
i mean if we are going to fully expand on it
Context from prxt0 #1122191 2026-06-22 14:47
this is what i think the point is - focusing on this event not talking about all work in general: Punk6529 approach: merge the CI step - have all PRs fail - spent 2h45mins to fix after prxt approach: spent 2h45mins to fix tests then merge the CI step to block if failures this is the entire point i was trying to pass on this morning - but ended up semi-irritating you
punk6529 Dev Team Chat #1122186 2026-06-22 14:45
we make similar decisions
punk6529 Dev Team Chat #1122185 2026-06-22 14:45
so that when I am not here
punk6529 Dev Team Chat #1122184 2026-06-22 14:45
the reason I am writing all this out is to explain my logic
punk6529 Dev Team Chat #1122183 2026-06-22 14:45
and I am not saying this to just hear myself talk. I could just say "please prioritize this, not that"
punk6529 Dev Team Chat #1122180 2026-06-22 14:45
so given all of this, obviously my POV is we should not kick the can down the road again and obviously not just on this PR, but on the whole 100x larger topics
punk6529 Dev Team Chat #1122177 2026-06-22 14:44
3/ given #1 and #2, my #1 priority is getting this in place
punk6529 Dev Team Chat #1122175 2026-06-22 14:44
2/ for this, we need extensivei, deep, broad, 'superhuman', 'unreasonable' amounts of testing
punk6529 Dev Team Chat #1122173 2026-06-22 14:43
1/ the way to achieve all our hopes and dreams is to accelerate, parallelize, agent-ize etc
punk6529 Dev Team Chat #1122172 2026-06-22 14:43
and so to wrap up this overly long text
punk6529 Dev Team Chat #1122171 2026-06-22 14:43
for this we need tests, tests, more tests, additional tests, then more tests, followed by more tests and so on
punk6529 Dev Team Chat #1122170 2026-06-22 14:42
the important point is that we have a system that we can trust to work autonomously
punk6529 Dev Team Chat #1122169 2026-06-22 14:42
and of course even this is not the important point
punk6529 Dev Team Chat #1122168 2026-06-22 14:41
and the amount of human time needed to write the prompt to fix 2 tests and to fix 1232435 tests is "exactly the same"
punk6529 Dev Team Chat #1122160 2026-06-22 14:39
because the only scarce thing on this team is human time
punk6529 Dev Team Chat #1122159 2026-06-22 14:39
is exactly the point of view I am trying to change
punk6529 Dev Team Chat #1122157 2026-06-22 14:39
you see, the fact that you think that the number of failing tests matters, that if it was 2 tests, it was logical to fix them but if it is 1232435 tests, it was not logical to fix them
punk6529 Dev Team Chat #1122156 2026-06-22 14:38
but on the 22nd of June, 2026, there is GPT 5.5 xhigh and Codex that CAN do this
punk6529 Dev Team Chat #1122155 2026-06-22 14:38
and fix them while "i was doing something else altogether"
punk6529 Dev Team Chat #1122153 2026-06-22 14:38
and you know, a year ago, there was no codex that could ground through 1232435 failing tests
punk6529 Dev Team Chat #1122152 2026-06-22 14:38
the can would be kicked down the road again and again and again
punk6529 Dev Team Chat #1122149 2026-06-22 14:37
and if I said "it is ok guys, no need to clean this up this morning"
punk6529 Dev Team Chat Reply #1122148 2026-06-22 14:37
yes because the tests and the repo have gotten out of line because at every point over the last years the team was not resolving this topic
Context from prxt0 #1122147 2026-06-22 14:37
my PR was changing one constant and reporting 1232435 tests failing
punk6529 Dev Team Chat #1122146 2026-06-22 14:37
whereas resolving the issue correctly took me a 5 minute prompt
punk6529 Dev Team Chat #1122144 2026-06-22 14:36
I remember this from earlier times in human history like "this morning"
punk6529 Dev Team Chat Reply #1122142 2026-06-22 14:36
because you wanted to roll back the CI gate as opposed to fixing the PR
Context from prxt0 #1122141 2026-06-22 14:36
i still dont understand what makes you think that i thought i should manually fix the failing tests i havent opened an IDE in months 🥹
punk6529 Dev Team Chat #1122140 2026-06-22 14:35
it took 2 hours and 45 minutes
punk6529 Dev Team Chat #1122139 2026-06-22 14:35
but also I did prove my point on the tests :slightly_smiling_face: