search

thebennybox-Community / Community-Compiler

MIT License
16 stars 5 forks source link

Removed redundant files from old documentation and expanded the main README.md file. #125

Closed jackieh closed 6 years ago

jackieh commented 6 years ago

The additional files, additional directory, and lack of pointers in the main README.md file created confusion when searching for the relevant documentation, so hopefully the nature of this project will be more straightforward to someone glancing at this repository.

davidgarland commented 6 years ago

Besides some inconsistencies with capitalization on headers (which can be fixed in a future PR), I think it looks good. Approved, on my end.

jorolf commented 6 years ago

I'm not sure wether the readme should mention the c++ compiler considering that were going to replace it with the kotlin one

jorolf commented 6 years ago

And could you update docs/readme.md and the other files with headers and links etc.

jackieh commented 6 years ago

The internal links at the top of *.md files have been fixed. I would very much prefer to not change anything about the choice of language implementing the first compiler (between C++ and Kotlin) in this pull request because it looks like there's no consensus regarding this yet in the Discord and I'm not the one who has any final say in this decision. It seems more appropriate for someone else to make that change in a subsequent pull request.