-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Better errror message for lists in config file #4914
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
Comments
@stsewd |
No yet, it depends on #4740, in we should think in a way of resolve this #4740 (comment) |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This isn't blocker anymore, but we still need to figure out a clean way of implementing this. And I'm not really sure adding that logic is worth it, we should just keep the current message. |
This will take care the person that implements it. I'm removing the
We can decide that when we see the changes in the PR propsed. |
After #4740 is merged, we will show the error message for lists as
list.{index}
, which isn't so clear, we should uselist[{index}]
instead.Raised in #4740 (comment)
The text was updated successfully, but these errors were encountered: