profile
viewpoint
If you are wondering where the data of this site comes from, please visit https://api.github.com/users/mosic/events. GitMemory does not store any data, but only uses NGINX to cache data for a period of time. The idea behind GitMemory is simply to give users a better reading experience.
Miloš Mošić mosic London, United Kingdom https://milosmosic.com/ Senior Software Engineer

MainframeHQ/mainframe-os 42

Mainframe OS

mosic/exdisque 15

Elixir client for Disque (https://github.com/antirez/disque), an in-memory, distributed job queue.

mosic/ansible-sumocollector 0

Ansible Role for SumoLogic Collector

mosic/ashes 0

A code generation tool for the Phoenix web framework

mosic/beets-check 0

Verify and store checksums in your beets library

mosic/calendar 0

date-time and time zone handling in Elixir

mosic/caramel 0

:candy: An Erlang backend to the OCaml compiler

mosic/cdp-pro-alerts 0

Backend for CDP PRO alerts

mosic/convex_sample 0

Sample application using Convex

mosic/credo 0

A static code analysis tool for the Elixir language with a focus on code consistency and teaching.

starteddoyensec/inql

started time in 18 days

startedsphaso/noether

started time in 22 days

pull request commentrrrene/credo

Add a check for IO.puts

Sorry for the late response - I was on vacation. That sounds good!

I updated the PR with the changes you proposed. Feel free to change the wording, wasn't sure if I should add more to the check description.

mosic

comment created time in a month

push eventmosic/credo

Miloš Mošić

commit sha 1499131df9e097052f285bbb3590d9d66e2a4cbd

Add a check for IO.puts

view details

push time in a month

pull request commentrrrene/credo

Add a check for IO.puts

That's a fair assesment, I meant to add it as an optional check. For example, in the project I'm working on, we use Logger statements anywhere, which automatically include the metadata etc.

IO.puts and IO.inspect obviously don't have all of the benefits logging statements do - but they're still useful for quick debugging, and sometimes they get left behind and caught in the PR review process.

Does that make sense? If it's not a common enough use-case, it's fine not to merge this.

mosic

comment created time in 2 months

PR opened rrrene/credo

Add a check for IO.puts

I was surprised to see that there was a check for IO.inspect and not for IO.puts, so I added one. It's almost an exact copy of the IO.inspect code.

Btw, thanks for your hard work on Credo! Saved me a bunch of time over the years.

+115 -0

0 comment

3 changed files

pr created time in 2 months

push eventmosic/credo

Miloš Mošić

commit sha 00cec0e618bb72408c287631f3a82f5dc12e2de6

Add a check for IO.puts

view details

push time in 2 months

create barnchmosic/credo

branch : add-io-puts

created branch time in 2 months

fork mosic/credo

A static code analysis tool for the Elixir language with a focus on code consistency and teaching.

http://credo-ci.org/

fork in 2 months

startedmarta-file-manager/marta-issues

started time in 2 months

startedachedeuzot/ueberauth_auth0

started time in 3 months