search

motet-a / yan

[DEPRECATED] Yet another norminette
6 stars 1 forks source link

This style checker is out of date #8

Open demaisj opened 6 years ago

demaisj commented 6 years ago

I'm just informing you that this norminette is currently out of date. There is a new coding style imposed for C projects since the start of the 2022 unix & c pool. You can check the latest version of the new coding style here: https://intra.epitech.eu/file/public/technical-documentations/epitech_coding_style.pdf

I'm not asking you to redo this norminette, I'm just here to tell you it is outdated. (maybe a mention in README.md would be useful?)

motet-a commented 6 years ago

Hello,

Thanks. At first glance, this new coding style is a huge step forward. No hard-limited function length, const-correctness, and constant global variables are well-known good practices for years. Honestly, I’m happy to deprecate this project.

However, does a new style checker exist? If not, updating yan doesn't look so hard to me since single-line macros (the main syntactic oddity) is still preserved in this new guide. Even if the style checks themselves require a big rewrite, the complicated parts (parsing) should only need minor updates (supporting switch statements and allowing variable declarations inside for statements is pretty easy).

demaisj commented 6 years ago

Since the coding style was officialy out yesterday, I don't think there is any style checker up to date for now on GitHub. Nevertheless, there is an official style checker (that seems to work better than the old), but we won't have access to it. The coding style is bound to maybe change, since some rules are non-justified or makes some pool tasks impossible to do, so I think it's the best bet to wait a week or two before doing anything.