On Tue, 16 Jan 2024 at 12:02, Jermain Horsman wrote:
> I did implement a new version, it is available on the mailing list,
> it should include all other feedback as well.
Thanks for persevering; the last version is good, so please re-submit
it without the RFC.
Once it lands we can think of
> Is comma (,) as a separator okay? Maybe : is better?
In my opinion a comma suggest a list, whereas a semicolon
suggests a relation, so I've opted to use ':'.
I did implement a new version, it is available on the mailing list,
it should include all other feedback as well.
Sincerely,
Jermain
On Mon, 8 Jan 2024 at 15:42, Jermain Horsman wrote:
All the new options need longer, more detailed descriptions.
> +parser.add_argument('--update', '-u',
> +action='store_true',
> +help='Updates an existing layers setup.')
"Updates an existing layer setup json
There are a few additional remarks that I'd like some input on.
> +logger.error("No repository specified. Please provide one
> using '--repository REPOSITORY'.")
> +raise Exception("No repository specified. Please provide one
> using '--repository REPOSITORY'.")
From: Jermain Horsman
This uses an existing setup-layers configuration and modifies one
or more repositories using a reference provided by the user.
This is a very minimal implementation, no validation of this reference
is done and it is left to the user to provide a valid value.