-
Notifications
You must be signed in to change notification settings - Fork 199
Allow maintainers to opt-out of tests #981
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
So what is the proposed design? Spinning some ideas:
(Maybe there should be aliases for singular/plural: |
I'm not sure if we want to use a I would prefer we just add metadata state to packages which maintainers can set from the "maintainers corner". |
How does this metadata state work? Where/how is it stored and modified? Could you point me to some source locations? |
State is stored using Acid-state in a number of distinct state components. I think the "preferred" path is probably a decent one to follow here -- create a new component storing this data (and perhaps some other data we anticipate letting maintainers set on packages while we're at it -- state migrations are possible but slightly irritating, so better to have more of what you want all at once on the first merge), then use acid-state to derive the storage methods for it, initialize the component on server startup, and allow it to be modified from the maintainers' corner., and consulted by hackage-server when it generates the page and also when the api is called that asks "ok, what do you want me (the builder) to build". |
PR #1124 lets us turn off tests for individual versions of a package. better still would also be allowing turning off tests for a package altogether. |
Some tests are prohibitive to run on hackage, or require special resources or packages not available. It would be nice to allow maintainers to choose to opt out of such things.
P.S. (2022-03-25 by @andreasabel):
As of today, you can opt out by uploading the documentation (via
cabal upload -d
) before the build/tests have run (#1037).The text was updated successfully, but these errors were encountered: