Skip to content
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

feat: use return error codes for easier integration in other projects #7

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

BoboTiG
Copy link
Contributor

@BoboTiG BoboTiG commented Jul 12, 2023

That way we can use that cool new tool into our automation without having to parse the STDIN output.

@tserg
Copy link
Owner

tserg commented Jul 13, 2023

Thanks for this. I don't foresee any integration for the moment, so I will just leave this PR open until the need arises.

@BoboTiG
Copy link
Contributor Author

BoboTiG commented Jul 13, 2023

I mean for now if I want to use Medusa in my projects, like I would do with flake8, I have to parse the output to be sure there was no issues. With that PR, I will know something "bad" was found, or not, thanks to the return code (it will stop the process with an error for instance).

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.

None yet

2 participants