Skip to content

Modern era update #3

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

am-kantox
Copy link

Updates and improvements:

  • all deps are up-to-date
  • .formatter.exs + all the files are formatted;
  • fixed spec for Matrix.{sub/2,add/2,do_binary_op/3} (dialyzer green.)
  • added graceful_shutdown: boolean() opts in calls to Expline.start/2 and Expline.start_link/2 (backward compatible,) to suppress gen_server crash logs on malformed input

Thank you for creating this!

Comment on lines +29 to +34
{graceful_shutdown, opts} = Keyword.pop(opts, :graceful_shutdown, false)

case GenServer.start(__MODULE__, {graceful_shutdown, [points]}, opts) do
{:error, {:shutdown, reason}} -> {:error, reason}
other -> other
end
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is (and below for start_link/2) the only significant change, everything else is formatter.

Aleksei Matiushkin and others added 2 commits November 27, 2023 09:26
* :version bump: for temporary private repo

* :version bump: for temporary private repo

* Merged curvature from lumaere

* 📎 version bump

* Defensive subtract (allowing `nil`)

* 0 ≠ 0.0

* Prepared for OTP27, ±0.0

---------

Co-authored-by: Larry Wu <lwu@drwholdings.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant