Skip to content
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

Merge script viewer #42

Open
wants to merge 233 commits into
base: dev
Choose a base branch
from
Open

Merge script viewer #42

wants to merge 233 commits into from

Conversation

RLKRo
Copy link
Member

@RLKRo RLKRo commented Nov 15, 2022

Description

Merge df_script_viewer.

Checklist

ruthenian8 and others added 7 commits April 4, 2023 18:39
Removed PYTHON3.9+ and PYTHON3.8- separation:
The changes between parser outputs are insignificant and the lines that caused the changes were present to test some edge cases. Instead of those lines new test cases are added to `test_complex`.

Removed unnecessary `TEST_CASES`. `TEST_CASES` should test DFFProject methods, not edge cases for parser. New tests are added to `test_complex` instead.

Removed separation between `conversions` and `to_python` test cases. They are now both in `complex_cases`.

Additionally:
- Added support for parsing unsupported statements from_dict and from_graph.
- Removed remove_prefix and remove_suffix
Change Actor location.

Replace Actor with Pipeline in `TEST_CASES`.
@ruthenian8 ruthenian8 force-pushed the merge/script_viewer branch from e145949 to 4128ff0 Compare April 4, 2023 17:25
@ruthenian8 ruthenian8 marked this pull request as ready for review April 7, 2023 11:45
@kudep kudep removed this from the Release 0.3.0 milestone Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants