You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hello, everybody! The long term goal of making hie-bios ready to merge has been achieved! CI is all green!
But before merging, I propose a day of review time!
Especially going through the README and fixing Documentation issues would be important in my opinion and to delete unused code and update documentation, except when there is a TODO or a specific goal to re-integrate the unused code again, e.g. HaRe code fragments.
I also would suggest that the time for review ends on Friday, 20.12.2019, unless someone wants to review but has no time till the weekend.
Also, it would be very nice if we find the time to document features that are disabled right now, but we want to fix in the next weeks, such as HaRe and goto-definition cross module.
Is this an acceptable deadline, @mpickering, @alanz, @bubba, @jneira? And whoever has the time and the will to review this behemoth pr.
Hello, everybody! The long term goal of making hie-bios ready to merge has been achieved! CI is all green!
But before merging, I propose a day of review time!
Especially going through the README and fixing Documentation issues would be important in my opinion and to delete unused code and update documentation, except when there is a TODO or a specific goal to re-integrate the unused code again, e.g. HaRe code fragments.
I also would suggest that the time for review ends on Friday, 20.12.2019, unless someone wants to review but has no time till the weekend.
Also, it would be very nice if we find the time to document features that are disabled right now, but we want to fix in the next weeks, such as HaRe and goto-definition cross module.
Is this an acceptable deadline, @mpickering, @alanz, @bubba, @jneira? And whoever has the time and the will to review this behemoth pr.
This issue is tightly coupled with #48.
The text was updated successfully, but these errors were encountered: