-
Notifications
You must be signed in to change notification settings - Fork 109
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consolidate flags from codegen into as_config #692
Conversation
LGTM and I actually want this soon so I can use the flag in the type checker and clean up the platform flag in async.ml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
let trace = ref false | ||
let verbose = ref false | ||
let print_depth = ref 2 | ||
let release_mode = ref false | ||
let compile_mode = ref ICMode | ||
let multi_value = ref false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought it was a feature that flags that only affect one module/library should be there. But consistency is the something something hobgoblin. So not a big deal either way.
|
## Changelog for motoko-base: Branch: next-moc Commits: [dfinity/motoko-base@8fd23fa7...a3dd04eb](dfinity/motoko-base@8fd23fa...a3dd04e) * [`cee15564`](dfinity/motoko-base@cee1556) feat: add `isReplicated` to `ExperimentalInternetComputer` ([dfinity/motoko-base#694](https://togithub.com/dfinity/motoko-base/issues/694)) * [`c9a08167`](dfinity/motoko-base@c9a0816) feat: add `Error.isRetryPossible`, a predicate for non-fatal messaging errors ([dfinity/motoko-base#692](https://togithub.com/dfinity/motoko-base/issues/692)) * [`6246b654`](dfinity/motoko-base@6246b65) feat: make `replyDeadline` an optional return type ([dfinity/motoko-base#693](https://togithub.com/dfinity/motoko-base/issues/693)) * [`1a381b99`](dfinity/motoko-base@1a381b9) Motoko 0.14.3 * [`756855bc`](dfinity/motoko-base@756855b) chore: amend the 0.14.3 changelog ([dfinity/motoko-base#697](https://togithub.com/dfinity/motoko-base/issues/697))
## Changelog for motoko-base: Branch: next-moc Commits: [dfinity/motoko-base@8fd23fa7...a3dd04eb](dfinity/motoko-base@8fd23fa...a3dd04e) * [`cee15564`](dfinity/motoko-base@cee1556) feat: add `isReplicated` to `ExperimentalInternetComputer` ([dfinity/motoko-base#694](https://togithub.com/dfinity/motoko-base/issues/694)) * [`c9a08167`](dfinity/motoko-base@c9a0816) feat: add `Error.isRetryPossible`, a predicate for non-fatal messaging errors ([dfinity/motoko-base#692](https://togithub.com/dfinity/motoko-base/issues/692)) * [`6246b654`](dfinity/motoko-base@6246b65) feat: make `replyDeadline` an optional return type ([dfinity/motoko-base#693](https://togithub.com/dfinity/motoko-base/issues/693)) * [`1a381b99`](dfinity/motoko-base@1a381b9) Motoko 0.14.3 * [`756855bc`](dfinity/motoko-base@756855b) chore: amend the 0.14.3 changelog ([dfinity/motoko-base#697](https://togithub.com/dfinity/motoko-base/issues/697))
This is a bit noisy PR that I wanted to leave out of #684. It makes flag handling less diverse. Mostly mechanical changes. No functional impact.