Skip to content

Feature/handle gundown#197

Merged
elbrujohalcon merged 8 commits into
inaka:masterfrom
mkrusemark:feature/handle_gundown
Oct 7, 2024
Merged

Feature/handle gundown#197
elbrujohalcon merged 8 commits into
inaka:masterfrom
mkrusemark:feature/handle_gundown

Conversation

@mkrusemark

Copy link
Copy Markdown
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

@elbrujohalcon elbrujohalcon left a comment

Copy link
Copy Markdown
Member

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.

Comment thread rebar.config Outdated
Comment thread src/shotgun.erl Outdated
Comment thread src/shotgun.erl Outdated
@elbrujohalcon

Copy link
Copy Markdown
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
Copy Markdown
Member

Fixed :)

@elbrujohalcon elbrujohalcon left a comment

Copy link
Copy Markdown
Member

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.

Comment thread src/shotgun.erl Outdated
Comment thread src/shotgun.erl Outdated
Comment thread test/http_server.erl Outdated
@mkrusemark

Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

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

@mkrusemark

Copy link
Copy Markdown
Contributor Author

Please tag the new version with 1.1.0.

@mkrusemark mkrusemark closed this Oct 7, 2024
@elbrujohalcon

Copy link
Copy Markdown
Member

Why have you closed this PR, @mkrusemark ?

@elbrujohalcon

Copy link
Copy Markdown
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
Copy Markdown
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