Reorder prior reading
Compare changes
Maintenance will be performed on git.ligo.org, containers.ligo.org, and docs.ligo.org on Tuesday 22 April 2025 starting at approximately 9am PDT. It is expected to take around 30 minutes and there will be several periods of downtime throughout the maintenance. Please address any comments, concerns, or questions to the helpdesk. This maintenance will be upgrading the GitLab database in order to be ready for the migration.
This MR reorders the set of things attempted when trying to parse the prior argument strings. The previous method was causing issues, e.g., the one identified in !617 (merged).
I hope the new order makes a bit more logical sense than the old way. A more descriptive error message is raised and I added a docstring so that future developers will hopefully follow better what is going on.
This will strip the r'' from any parameters that are passed as raw strings, e.g., r'foo'
will just render as foo
.
@sylvia.biscoveanu can you test this on the prior you had issues with.