Skip to content

Fix property description#12

Closed
yhoiseth wants to merge 1 commit intoplatformsh:masterfrom
yhoiseth:patch-3
Closed

Fix property description#12
yhoiseth wants to merge 1 commit intoplatformsh:masterfrom
yhoiseth:patch-3

Conversation

@yhoiseth
Copy link
Contributor

routes() does not work, but routes does:

./manage.py shell
Python 3.7.0 (default, Aug 20 2018, 13:43:00) 
[GCC 6.3.0 20170516] on linux
Type "help", "copyright", "credits" or "license" for more information.
(InteractiveConsole)
>>> from platformshconfig import Config
>>> config = Config()
>>> config.routes()
Traceback (most recent call last):
  File "<console>", line 1, in <module>
TypeError: 'dict' object is not callable
>>> config.routes
{'https://www…

@kotnik kotnik requested a review from NickAnderegg March 19, 2019 14:18
@chadwcarlson
Copy link
Contributor

We have routes as a method in the other config-readers, and it looks like the inclusion of a @property decorator created this error. I have modified this in #13

@chadwcarlson chadwcarlson mentioned this pull request Mar 20, 2019
@Crell Crell closed this in #13 Mar 20, 2019
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.

2 participants