Skip to content

Added FDL support#127

Open
AryanP281 wants to merge 5 commits intoCollaborativeStateMachines:developfrom
AryanP281:develop
Open

Added FDL support#127
AryanP281 wants to merge 5 commits intoCollaborativeStateMachines:developfrom
AryanP281:develop

Conversation

@AryanP281
Copy link
Copy Markdown

Pull Request

Description

Added FDL schemas which can be compiled into classes (or other analogous logical structures) in the supported languages. The fory compiler is used for this purpose and needs to be installed as a python package using PIP.

Due to bug(s) in the fory compiler, the java classes generated using the compiler have incorrect syntax and hence need to be manually corrected (after generation using the gradle task "generateForyTypes"). Other supported languages have been not tested by me for this issue.

Type of change

Please delete options that are not relevant.

Breaking change (fix or feature that would cause existing functionality to not work as expected)

How has this been tested?

Using existing unit and integration tests. Also added new tests to test the following -

  1. Serialization & deserialization of List of ContextVariables
  2. Serialization & deserialization of List of Events
  3. Serialization & deserialization of an empty list

All tests pass.

Checklist:

  • [Y] My code follows the style guidelines of this project
  • [Y] I have performed a self-review of my own code
  • [Y] I have commented my code, particularly in hard-to-understand areas
  • [Y] I have made corresponding changes to the documentation
  • [Y] My changes generate no new warnings
  • [Y] I have added tests that prove my fix is effective or that my feature works
  • [Y] New and existing unit tests pass locally with my changes
  • [Y] Any dependent changes have been merged and published in downstream modules

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.

1 participant