-
Notifications
You must be signed in to change notification settings - Fork 27
Description
These are some observations where it looks like output feed from the CLI could be cleaned up.
I ran pretext build on the showcase article, and the CLI shows:
warning: Project's CLI version could not be detected from `requirements.txt`.
warning: Try `pretext init --refresh` to produce a compatible file.
OK, so I run pretext init --refresh. This time:
warning: Project's CLI version could not be detected from `requirements.txt`.
warning: Try `pretext init --refresh` to produce a compatible file.
warning: The `pretext init --refresh` command will be deprecated in a future version. You can use `pretext update --force` instead.
-
Since I am running
pretext init --refresh, it's odd to be told that I should runpretext init --refresh. -
Also now there is the note about this being deprecated in the future. Should the first message have directly suggested I use
pretext update --force? -
Now maybe the biggest concern. This clobbered the
project.ptxfile. It removed all the targets and replaced them with just the default "web" and "print" targets. Since this was the showcase article, it's easy to see the diff and restore those targets. And I see thatproject.ptx.bakexists now. But I feel like the casual suggestion to usepretext init --refreshin the first place should come with a warning that this would kill all the carefully designed targets. Or there should be a different way to narrowly address the originalrequirements.txtissue that does not involve messing with theproject.ptxfile. -
Lastly, it seems there just is no
requirements.txtfile for the showcase article in the pretext repo in the first place. Is this a file that should be added to the repo? Or more to the point, what should be done to the pretext repo so that runningpretext buildjust works cleanly out of the box?