Skip to content

Remove release.py and Rakefile #457

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

Conversation

victorolinasc
Copy link
Contributor

@victorolinasc victorolinasc commented Jun 20, 2020

This project is centered around providing an Emacs Lisp major mode to the programming language
Elixir. It is a higher maintenance burden to demand Python and Ruby on the developer machine.

I've update the wording on how to test too. If this is to be left out I can simply reset that. There is, though, a fix on the link for Cask installation taken from the Cask guides.

Fixes #457

This project is centered around providing an Emacs Lisp major mode to the programming language
Elixir. It is a higher maintenance burden to demand Python and Ruby on the developer machine.

Also, this PR revamps the CI (which used the Rakefile) to use
https://github.com/purcell/nix-emacs-ci . This is a more recent approach to many Emacs versions on
Travis.
@victorolinasc victorolinasc self-assigned this Jun 20, 2020
@victorolinasc
Copy link
Contributor Author

Initially I wanted to update Elixir and Emacs versions of the CI here too since I was editing travis.yml but I've noticed the test-suite is failing. I'll try to take a look at that in another PR

Copy link
Contributor

@axelson axelson left a comment

Choose a reason for hiding this comment

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

This looks like a good simplification! ❤️

@axelson axelson merged commit bd417aa into elixir-editors:master Jun 21, 2020
J3RN pushed a commit to J3RN/emacs-elixir that referenced this pull request Apr 24, 2021
* remove legacy io_request handlers

we don't support OTP < R15B

* rescue MatchError in :int calls

Fixes elixir-editors#455

* make output device better conform to erlang I/O protocol

see https://erlang.org/doc/apps/stdlib/io_protocol.html for details

* return WireProtocol.send error to the caller

no need to IO.warn if write fails

* we are redirection stderr to stdout, use stdout as underlying device

* inspect error

* monitor debugged processes

add test for mix task exit
Fixes elixir-editors#454

* avoid debugger crashes when handling requests for no longer existing thread, frame and variable ids

Fixes elixir-editors#452

* add test

* forbid changes of underlying device opts

* refactor and add tests coverage to invalid requests

* Map.pop! is available since elixir 1.10

* run formatter
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.

2 participants