rossvideo / Catena

Other
6 stars 2 forks source link

we should move the cpp codegen files into their own folder #246

Open mejohnnaylor opened 4 weeks ago

mejohnnaylor commented 4 weeks ago

at the moment all the codegen files are in ${workspaceRoot}/tools/codegen. The codegen app is actually designed to generate code for a number of languages, although only c++ is currently supported. So, the c++ specific files should move to a sub-folder, cpp under codegen. Apart from moving a few files (params.js, cppgen.js, device.js) a few require statements will need to be changed.

gRid08 commented 3 weeks ago

Hi @mejohnnaylor I would like to work on this.. should we move cppctor.js to the cpp subfolder

mejohnnaylor commented 3 weeks ago

H Grid08 - yes please, all the .js files apart from index.js should go in a cpp folder and the require statements updated to reflect this.

Welcome to the project!

John

mejohnnaylor commented 3 weeks ago

out of interest - how did you find out about this project, and what's your motivation to help us out?

gRid08 commented 3 weeks ago

Actually this is my first open-source contribution and I was looking for projects with microservices architecture and good-first issues to begin with. I am eager to learn and contribute, and I hope you can guide me through the process. Looking forward to working with you.

gRid08 commented 3 weeks ago

250 pr do tell if any changes