the sentence in the app page's Volume Mounts section for an app with no volume mounts
the same sentence in each such app's deploy log, and the NoVolumesWarning constant behind both
the two tests for them
the README sentence that a deploy or rollback keeps only the files in the app's volume mounts
its TODO.md entry
One conflict, in TODO.md: later entries were added above the removed one. I kept those (#239, #253, #255) and dropped only the one for #248.
The environment variable hint from #255 keeps its alert-warning class. That class is defined in static/css/input.css and is in the committed static/css/tailwind.css, which the build does not regenerate, so removing the Volume Mounts sentence does not take it out of the stylesheet.
Model: opus-5-5
sneak asked for this in https://git.eeqj.de/sneak/upaas/pulls/254#issuecomment-109763: "this is stupid, revert this entire commit".
This reverts `cc786a1e` (https://git.eeqj.de/sneak/upaas/issues/248) in full. It removes:
- the sentence in the app page's Volume Mounts section for an app with no volume mounts
- the same sentence in each such app's deploy log, and the `NoVolumesWarning` constant behind both
- the two tests for them
- the README sentence that a deploy or rollback keeps only the files in the app's volume mounts
- its `TODO.md` entry
One conflict, in `TODO.md`: later entries were added above the removed one. I kept those (https://git.eeqj.de/sneak/upaas/issues/239, https://git.eeqj.de/sneak/upaas/issues/253, https://git.eeqj.de/sneak/upaas/issues/255) and dropped only the one for https://git.eeqj.de/sneak/upaas/issues/248.
The environment variable hint from https://git.eeqj.de/sneak/upaas/issues/255 keeps its `alert-warning` class. That class is defined in `static/css/input.css` and is in the committed `static/css/tailwind.css`, which the build does not regenerate, so removing the Volume Mounts sentence does not take it out of the stylesheet.
Model: opus-5-5
This reverts commit cc786a1e84, at the
owner's request. It removes the app page sentence for an app with no
volume mounts, the matching deploy log line and its constant, their
tests, the README sentence and the TODO.md entry. The environment
variable hint keeps its alert-warning class, which is in the committed
stylesheet.
Model: opus-5-5
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
sneak asked for this in #254 (comment): "this is stupid, revert this entire commit".
This reverts
cc786a1e(#248) in full. It removes:NoVolumesWarningconstant behind bothTODO.mdentryOne conflict, in
TODO.md: later entries were added above the removed one. I kept those (#239, #253, #255) and dropped only the one for #248.The environment variable hint from #255 keeps its
alert-warningclass. That class is defined instatic/css/input.cssand is in the committedstatic/css/tailwind.css, which the build does not regenerate, so removing the Volume Mounts sentence does not take it out of the stylesheet.Model: opus-5-5
Review passed.
Model: opus-5-5