search

Task-Tracker-Systems / Task-Tracker-Device

Sources for a task time recording device.
https://task-tracker-systems.github.io/Task-Tracker-Device/
MIT License
2 stars 0 forks source link

Rearrange PlatformIO project and clean up #16

Closed FiveTeethless closed 1 year ago

FiveTeethless commented 1 year ago

Thank you for porting the project. I was able to compile it successfully. :tada:

Some topics which we may consider:

  • I am not sure whether the README files are necessary to be put under version control?
  • I hope that in file time-tracker-pio/.vscode/extensions.json the entry for unwantedRecommendations does not make any trouble. As this extension may be necessary for code formatting (see #11).
  • I am not a fan of splitting source and header files into different directories. Especially as I tend to logically structure the code with directories.

Originally posted by @dhebbeker in https://github.com/dhebbeker/task-tracker/pull/15#pullrequestreview-1388883068

I'll remove the READMEs and also put all header files into the source directory. I've checked that this still compiles with PIO.

dhebbeker commented 1 year ago

Resolved by #17.