Purge the examples folder
I just spent too much time tracking down a bug which appeared when changing the version from 0.5.1-DEV to 0.5.1. The example folder files requires include code which is necessary for the non-example tests to pass. But the examples folder is only incuded, if is_dev is true.
On Fri, Feb 12, 2021 at 02:26:22AM -0800, thofma wrote:
I just spent too much time tracking down a bug which appeared when changing the version from
0.5.1-DEVto0.5.1. The example folder code requires code which is necessary for the tests to pass. But the examples folder is only incuded, ifis_devis true.
Move the stuff that is automatically tested in dev into experimental? Where it is always included?
You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/oscar-system/Oscar.jl/issues/280
https://github.com/oscar-system/Oscar.jl/blob/master/src/Oscar.jl#L218-L226
I don't understand why this was not moved to experimental.
Essentially this is a purely "experimental" implementation from a Fach Praktikum. Wolfram is waiting for the link to the Singular library Smilar: All of Delphine's stuff should be there, but that needs coordination with her to avoid merge conflicts and frustartion
@delphinepol Once your recent PR is merged, could you make a PR to move your things from examples to experimental? That would be great, thanks.
OK !
I think that most of the code will probably be broken right now as all of this stuff is not covered by CI
Triage:
@fieker likes the examples folder. Someone (TM) would need to go through all of the examples code, fix it, remove stuff that is now in src or experimental, ...
Maybe @thofma or @fingolfin can be there next triage and bring up their opinion?