-
Notifications
You must be signed in to change notification settings - Fork 16
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
Support of DATE
format
#24
Comments
hi @lojonas |
Hello !
Is there any plans to add the date format to other parts of the library in the future? |
startet implementing it: |
* support for DATE * fix typos #24 * Issue 24 (#29) * Add "REAL", "DATE" and "SIMPLE" data types Add dictionary entries for "REAL", "DATE" and add "SIMPLE" data type * Issue 24 - Support vor values without div tags (#31) * Add "REAL", "DATE" and "SIMPLE" data types Add dictionary entries for "REAL", "DATE" and add "SIMPLE" data type * Fix typo * Tabs to spaces * More tabs to spaces * add support for elements without div tag Co-authored-by: ebroecker <eduard@gmx.de>
I plan to read in a
reqif
file with your parser and then edit it accordingly with additional information. The issue I came across is that the `ATTRIBUTE-DEFINITION-DATE' and its datatype conter parts are not defined, e.g;pyreqif/src/pyreqif/rif.py
Line 180 in ce9dd62
pyreqif/src/pyreqif/rif.py
Line 219 in ce9dd62
Do you plan to integrate them? Or can you give me a hint where I need to work to integrate them?
I have tried to just adapt the if statements where the checks are conducted (see above) and have changed the dicts
pyreqif/src/pyreqif/rif.py
Line 42 in ce9dd62
to include the DATE attributes and map them to a string? Maybe there is a better datatype in python, but for me it is not necessary, as I do not plan to edit the dates.
The text was updated successfully, but these errors were encountered: