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

Fix filtering_lines parameter to also apply when inline_mode is false #33

Merged
merged 3 commits into from
Apr 19, 2022

Conversation

petitJAM
Copy link
Contributor

Fixes an issue with #30 / #18

I upgraded to Ruby 3 since I couldn't get 2.3.5 to install on my machine (I didn't try very hard 😅). I'm not noticing any problems, so I think it's good.

@petitJAM petitJAM requested a review from barbosa as a code owner April 18, 2022 18:45
@barbosa barbosa merged commit 447b63c into loadsmart:master Apr 19, 2022
@barbosa
Copy link
Contributor

barbosa commented Apr 19, 2022

That should be fine. Thank you for your contribution, @petitJAM. Appreciate it.

@petitJAM petitJAM deleted the fix-filtering-lines branch April 22, 2022 15:26
@petitJAM petitJAM restored the fix-filtering-lines branch April 22, 2022 16:20
@petitJAM
Copy link
Contributor Author

@barbosa Not to be annoying or rush you or anything, but when would you expect to publish a new version of the gem? I'd like to set myself a reminder to update our system to the official release since I'm installing from my branch currently.

@barbosa
Copy link
Contributor

barbosa commented Apr 22, 2022

just published it @ https://rubygems.org/gems/danger-android_lint/versions/0.0.10

@petitJAM petitJAM deleted the fix-filtering-lines branch April 22, 2022 17:15
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