search

Wunderbyte-GmbH / moodle-mod_datalynx

Repository for the Datalynx Module for Moodle
7 stars 5 forks source link

Check "remove merge" comments in code #144

Open michaelpollak opened 4 years ago

michaelpollak commented 4 years ago

I just stumbled again over this tag in the code, should check the reason it is there and if we can remove it.

    // YOU
    // *MUST*
    // REMOVE
    // THIS
    // MERGE!

Noticed here: https://github.com/Wunderbyte-GmbH/moodle-mod_datalynx/blob/7884d4c83cda66430d08684feac2081b46f32ac6/field/field_class.php#L304

michaelpollak commented 3 years ago

This comment is seven years old, obviously does not create any issues. I'll remove it for clarity. https://github.com/Wunderbyte-GmbH/moodle-mod_datalynx/commit/896183a32d135ce5e9937de50d43c8979822ee41