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.

Sign up for GitHub

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

Jump to bottom

Feat/colang2 alpha migration script #606

Closed
wants to merge 7 commits into from
Closed

Feat/colang2 alpha migration script #606

wants to merge 7 commits into from

Conversation

Pouyanpi
Copy link
Collaborator

@Pouyanpi Pouyanpi commented Jul 5, 2024 •
edited
Loading

This PR allows colang2.0-alpha migration. It also refactors the migration.py and adds from_version to CLI.

Remaining Tasks:

  • Should we save the old version of the co files? It currently does with 'v2-alpha.co' extension. Decision: do not save
  • Remove ccl.co and core_flow_library.co, are there more to add?
  • import modules from Colang Standard Library

@Pouyanpi Pouyanpi added the status: in progress Issues that are currently being worked on. label Jul 5, 2024
@Pouyanpi Pouyanpi added this to the v0.10.0 milestone Jul 5, 2024
@Pouyanpi Pouyanpi requested a review from drazvan July 5, 2024 12:22
@Pouyanpi Pouyanpi self-assigned this Jul 5, 2024
@Pouyanpi Pouyanpi added status: waiting confirmation Issue is waiting confirmation whether the proposed solution/workaround works. and removed status: in progress Issues that are currently being worked on. labels Jul 9, 2024
@Pouyanpi Pouyanpi force-pushed the feat/colang2-alpha-migration-script branch from f49fb9f to cb621c7 Compare July 10, 2024 14:13
@Pouyanpi
feat(migration): add migration script for colang 2.0-alpha and refactor
24bf60d 
This commit introduces a new migration script that helps in migrating colang (.co) files and config (.yaml/.yml) files from older versions to the latest version.
The script supports migration from "1.0" and"2.0-alpha" versions to the latest version.
The script also validates the migrated files to ensure they arecorrect.
@Pouyanpi
feat(cli): add from_version argument
65f2fd9
@Pouyanpi
docs(migration): update docstrings
0d76adb
@Pouyanpi
refactor(migration): rename functions and update string quotes
4b6a8af
@Pouyanpi
feat(migration): add support for CSL imports and remove obsolete files
d7a27f7
@Pouyanpi
refactor(migration): refactor file reading in conversion functions
ffc4e60 
The conversion functions convert_colang_2alpha_syntax and
convert_colang_1_syntax have been refactored to accept a list of lines
instead of a file path
@Pouyanpi
test(migration): add tests for colang 2alpha syntax conversion
585999e
@Pouyanpi Pouyanpi force-pushed the feat/colang2-alpha-migration-script branch from cb621c7 to 585999e Compare July 11, 2024 10:47
@drazvan
Copy link
Collaborator

drazvan commented Aug 22, 2024

Closing this as already included in feature/colang-2.0-library.

@drazvan drazvan closed this Aug 22, 2024
@drazvan drazvan mentioned this pull request Aug 22, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@drazvan drazvan Awaiting requested review from drazvan

Assignees

@Pouyanpi Pouyanpi

Labels
status: waiting confirmation Issue is waiting confirmation whether the proposed solution/workaround works.
Projects
None yet
Milestone
v0.10.0
Development

Successfully merging this pull request may close these issues.
2 participants
@Pouyanpi @drazvan