stabilize path operations, optimize remove_comments, add initial sample for documentation #13
Conversation
…le for documentation
test-fullautomation
requested review from
HolQue,
milanac030988,
namsonx,
ngoan1608 and
tranhoangnguyen193
|
Hi Son,
|
|
Hello Thomas, Thank you, |
|
Hi Son, |
sample/sample.py
Outdated
|
|
||
| prepro=JsonPreprocessor.CJsonPreprocessor() | ||
|
|
||
| data=prepro.jsonLoad(".\json\json_with_comment.json") |
There was a problem hiding this comment.
Hi Son,
important is that this use-case is working.
We assume that most of the users will not require this for robotframework, but for any other python project.
Usually in this case the configuration data is either relative to the project, or refrerenced by means of an environment variable.
Thank you,
Thomas
| sContent=file.read() | ||
| file.close() | ||
|
|
||
| pattern = re.compile(r'//.*?$|/\*.*?\*/|\'(?:\\.|[^\\\'])*\'|"(?:\\.|[^\\"])*"', re.DOTALL | re.MULTILINE) |
There was a problem hiding this comment.
This allows /* */ multiline comments, too.
- add annotations to functions
- absolute and relative paths are allowed for main file as well as [import] file.
- %envvariable% and {envvariable} syntax are allowed in paths
- [import] is now relative to the current file, instead of relative to the base file. This allows shorter paths and it allows to move whole sets of json files to other locations. Only the path to root file of a subset need to be changed in this case.
- all docstrings are now in rst syntax and maintained
- lot of documenation added
- f syntax is use for all exception strings.
- unified style of all exception strings.
|
Must be tested and then merged together with |
No description provided.