Skip to content
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

Allow specify index-state per repository #6597

Merged
merged 2 commits into from
Mar 21, 2020
Merged

Conversation

phadej
Copy link
Collaborator

@phadej phadej commented Mar 21, 2020

One concern I have is the syntax.

v2-update uses comma :

hackage.haskell.org,HEAD
hackage.haskell.org,2020-03-16T01:24:50Z
hackage.haskell.org,@123123123

Yet, I don't want comma in configuration files to mean anything else then list separator,
therefore I took colon, which we use e.g. in allow-newer and other "namespace" cases:

hackage.haskell.org:HEAD
hackage.haskell.org:2020-03-16T01:24:50Z
hackage.haskell.org:@123123123

This is a bit unfortunate, as timestamp has colons in it.

Another alternative would be to use @, note double @ when using posix-timestamp.

hackage.haskell.org@HEAD
hackage.haskell.org@2020-03-16T01:24:50Z
hackage.haskell.org@@123123123

I'll merge the colon version; this can be changed before 3.4 is released.
Perfectly, v2-update would use (or at least accept) the colons too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant