Skip to content

backend/sdoc/grammar: Add GRAMMAR for REFS-TypeValue variant#754

Merged
stanislaw merged 1 commit intostrictdoc-project:mainfrom
GGBeer:GEF-TypeValue
Dec 8, 2022
Merged

backend/sdoc/grammar: Add GRAMMAR for REFS-TypeValue variant#754
stanislaw merged 1 commit intostrictdoc-project:mainfrom
GGBeer:GEF-TypeValue

Conversation

@GGBeer
Copy link
Copy Markdown
Contributor

@GGBeer GGBeer commented Dec 2, 2022

Adding ParentReqReference and FileReference TypeValue-Grammar for GRAMMAR-REFS instead of String.
Note: Adding new ExternalReference and updating the Documentation will be done in a separate step.

Copy link
Copy Markdown
Collaborator

@stanislaw stanislaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent work, let's just polish some edges, and please get the branch to pass all tests.

Comment thread strictdoc/backend/sdoc/models/constants.py Outdated
Comment thread strictdoc/backend/sdoc/error_handling.py Outdated
Comment thread strictdoc/backend/sdoc/grammar/grammar.py Outdated
Comment thread strictdoc/backend/sdoc/models/reference.py
Comment thread strictdoc/backend/sdoc/models/type_system.py Outdated
Comment thread tests/unit/strictdoc/backend/sdoc/test_dsl_passthrough.py Outdated
@GGBeer
Copy link
Copy Markdown
Contributor Author

GGBeer commented Dec 5, 2022

Still have to rebase the last commits.
Just wanted to make sure that the CI itest complete without failures.

@stanislaw
Copy link
Copy Markdown
Collaborator

stanislaw commented Dec 7, 2022

@GGBeer,

@mettta and I are merging a big branch with some server and web frontend work just now. This is done to avoid big branches and accumulating conflicts. After I am done with some follow-up work related to that branch, I will get to review this your PR again - should do it today. Thank you for your patience.

@stanislaw
Copy link
Copy Markdown
Collaborator

General comment: I would be curious to learn which problems with itests did you encounter on Windows?

@GGBeer
Copy link
Copy Markdown
Contributor Author

GGBeer commented Dec 7, 2022

Must setup a new clean testsystem without any old python, pycharm, pip modules or installed in root vs. user space or whatever which could mess up the itest runs.
Pytest Unit-Tests run as expected, For iTests I have to execute the lit statement separately.

stanislaw
stanislaw previously approved these changes Dec 7, 2022
Copy link
Copy Markdown
Collaborator

@stanislaw stanislaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hereby approved. Please rebase it one last time and I will merge it.

…arElementFieldReference with ReferenceTypes

Add primitive ReferenceType-as-String handling for reqIf and Excel.
Update and adding some unit- and itests.
FileReference tests RegExpr for differing Linux/Windows OS-Path Normalisation
Copy link
Copy Markdown
Collaborator

@stanislaw stanislaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for addressing all review comments!

@stanislaw stanislaw merged commit bc4b73a into strictdoc-project:main Dec 8, 2022
@stanislaw
Copy link
Copy Markdown
Collaborator

Do you need a pip package release soon, or you are fine working in a developer mode for now?

I need a couple of days to polish and document the new server-related work before I release. Would you mind adding the documentation for the pieces of work that you have contributed? The target document is 01_user_guide and you don't have to be very verbose, just follow along the existing parts about the markup and the custom grammar.

Also note that I have added a separate https://strictdoc.readthedocs.io/en/latest/strictdoc_11_developer_guide.html. Let me know if you would like any further information to be there to simplify your or others' learning about the development workflow.

Thank you.

@GGBeer
Copy link
Copy Markdown
Contributor Author

GGBeer commented Dec 8, 2022

Working in a developer mode is fine for me.
I will add some docs into the user guide next. After that I will start a new ExternalReference branch.

Thx,
Guus

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