Conversation
alexhutman
force-pushed
the
linting
branch
6 times, most recently
from
ca4a871 to
784cb05
Compare
Tests shouldn't be included in the sdist nor installation.
Causing too many issues for little gain. If there's a dummy package, maybe we can use that, but it should be known that Sage is needed.
alexhutman
force-pushed
the
linting
branch
17 times, most recently
from
ca0830b to
e979d55
Compare
|
Hmm... currently running into an OOM issue during linting. Not really sure why, I'd think that building the wavefront code might require more memory. Might have to separate the linting workflow... |
…M error" This reverts commit bd9c6ac. Pruning didn't help... https://github.com/alexhutman/ZeroForcingNumber/actions/runs/10181002525/job/28160447440?pr=12
This reverts commit 941883e. Didn't work
|
Okay I think that OOM issue will be resolved. According to this, |
|
@ldeaett Sorry for the noise, thought I was ready but had to tweak some things. I think it's good now whenever you're ready. Just please don't merge yet -- next time we meet, we can do a working session where I show you how I'd clean up these commits a bit more before merging to master. |
Cancel that @ldeaett, you can merge whenever you approve. Just please rebase and merge! The commits could be worse... |
2 participants
Lint Cython files