Skip to content

Conversation

@mkrusemark
Copy link
Contributor

Added support for reconnection via a shotgun:reopen call.
Testsuites had to be modified because cover didn't work with latest erlang version.
Rebar3 crashed with rich compiler messages.

mkrusemark added 2 commits October 2, 2024 11:23
Copy link
Member

@elbrujohalcon elbrujohalcon left a comment

Choose a reason for hiding this comment

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

It's not bad but you seem to be using tabs for indentation and that breaks code style.
If you, instead of realigning everything manually, would like to add rebar3_format to this project and format the code according to its rules, that would be perfectly fine, too.

@elbrujohalcon
Copy link
Member

Hummm… In order to see only the relevant changes here, I applied the formatter independently in #198 … but it looks I didn't do it correctly, did I? 🤔

@elbrujohalcon
Copy link
Member

Fixed :)

Copy link
Member

@elbrujohalcon elbrujohalcon left a comment

Choose a reason for hiding this comment

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

A few comment removals, but everything else looks fine.

@mkrusemark
Copy link
Contributor Author

Hummm… In order to see only the relevant changes here, I applied the formatter independently in #198 … but it looks I didn't do it correctly, did I? 🤔

@mkrusemark
Copy link
Contributor Author

That's interesting. I added the plugin without any specific options, see rebar.config.

@mkrusemark
Copy link
Contributor Author

Please tag the new version with 1.1.0.

@mkrusemark mkrusemark closed this Oct 7, 2024
@elbrujohalcon
Copy link
Member

Why have you closed this PR, @mkrusemark ?

@elbrujohalcon
Copy link
Member

I'm confident it's a good change. I'll reopen and merge it.

@elbrujohalcon elbrujohalcon reopened this Oct 7, 2024
@elbrujohalcon elbrujohalcon merged commit ac6d937 into inaka:master Oct 7, 2024
@elbrujohalcon
Copy link
Member

Please tag the new version with 1.1.0.

Done: https://hex.pm/packages/shotgun/1.1.0

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