Skip to content
This repository has been archived by the owner on Jul 31, 2024. It is now read-only.

Set to disabled as default (ie opt-in to TruncatedStacktraces, instead of opt-out) #24

Merged
merged 4 commits into from
Mar 23, 2023

Conversation

sjdaines
Copy link
Contributor

Change default setting of disabled option to true, so package is disabled by default and requires a setting in LocalPreferences.toml to enable.

See #22, #23 enabling TruncatedStacktraces can cause significant slowdowns (minutes) in startup, and the LocalPreferences.toml mechanism to disable is not as straightforward as just adding the setting.

sjdaines and others added 3 commits March 22, 2023 15:07
Change default setting of disabled option to true, so package is disabled by default and requires a setting in
LocalPreferences.toml to enable.

See #22, #23
enabling TruncatedStacktraces can cause significant slowdowns (minutes) in startup.
Now that disable = true is the default, need to explicitly enable before running tests
@codecov
Copy link

codecov bot commented Mar 22, 2023

Codecov Report

Merging #24 (50d0ab2) into master (6b860aa) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #24   +/-   ##
=======================================
  Coverage   46.66%   46.66%           
=======================================
  Files           1        1           
  Lines          15       15           
=======================================
  Hits            7        7           
  Misses          8        8           
Impacted Files Coverage Δ
src/TruncatedStacktraces.jl 46.66% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ChrisRackauckas ChrisRackauckas merged commit eb05297 into SciML:master Mar 23, 2023
@sjdaines sjdaines deleted the opt_in branch March 23, 2023 15:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants