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

Cpp translation - initial generation of C++ files #33

Merged
merged 8 commits into from
Oct 31, 2023
Merged

Conversation

tanaya-mankad
Copy link
Contributor

@tanaya-mankad tanaya-mankad commented Oct 24, 2023

There are a couple of distinct changes in this branch, the most major of which is the addition of a cpp (headers only so far) translator that exports .h files to a cpp subdirectory of the build folder.

In addition,

  1. pylint warnings have started to be fixed in various python files, including files in the doc directory.
  2. Some instances of os.path have been replaced with pathlib Path.

I wanted to get this much work merged before fleshing out the following items:

  1. Using the new SchemaFile if possible for the cpp translation
  2. Add cpp implementation file generation
  3. More pylint fixes and the addition of a CI check for them
  4. Complete pathlib transition.

@tanaya-mankad tanaya-mankad added the enhancement New feature or request label Oct 24, 2023
@tanaya-mankad tanaya-mankad self-assigned this Oct 24, 2023
@tanaya-mankad
Copy link
Contributor Author

The latest checkin contains CPP code generation.

@tanaya-mankad tanaya-mankad changed the title Cpp translation - initial generation of header files Cpp translation - initial generation of C++ files Oct 30, 2023
@nealkruis nealkruis merged commit 6e251ce into main Oct 31, 2023
12 checks passed
@nealkruis nealkruis deleted the cpp-translation branch October 31, 2023 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants