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
Make sure the style and terminology is consistent with TypeDB
TypeDB is the reference implementation which we should adhere to.
Make the logic that parses the command terser and stricter, and with minimal boilerplate
Please take a look at this Demonstrational PR on how much effort it takes to add relatively simple functionality. The parser is not strict and error prone, and it requires a lot of boilerplate. There are some nuances in how we should best approach the refactor, which we should discuss verbally.
The text was updated successfully, but these errors were encountered:
If we still need a way to escape an inner CLI without exiting completely, we can replace the boolean that is passed around for success with an Enum which will be more meaningful and easy to read
lolski
pushed a commit
to lolski/typedb-console
that referenced
this issue
Jun 9, 2021
## What is the goal of this PR?
Speed up builds by utilizing remote caching provided by BuildBuddy.
## What are the changes implemented in this PR?
Reenable build caching (without remote execution) on all CI jobs
Make sure the style and terminology is consistent with TypeDB
TypeDB is the reference implementation which we should adhere to.
Make the logic that parses the command terser and stricter, and with minimal boilerplate
Please take a look at this Demonstrational PR on how much effort it takes to add relatively simple functionality. The parser is not strict and error prone, and it requires a lot of boilerplate. There are some nuances in how we should best approach the refactor, which we should discuss verbally.
The text was updated successfully, but these errors were encountered: