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.

Sign up for GitHub

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

Jump to bottom

[unblock build] Fix linting in monitors library #18

Merged

nzlosh merged 1 commit into StackStorm-Exchange:master from abhishek-j16:ajagannath/fixlint

Sep 30, 2024

Contributor

abhishek-j16 commented Sep 27, 2024

Build is failing because of the linting issue and hence blocking new package version release for the commit: #17

/home/runner/work/stackstorm-datadog/stackstorm-datadog/pack/actions/lib/monitors.py:22:45: E231 missing whitespace after ','
/home/runner/work/stackstorm-datadog/stackstorm-datadog/pack/actions/lib/monitors.py:23:53: E231 missing whitespace after ','
/home/runner/work/stackstorm-datadog/stackstorm-datadog/pack/actions/lib/monitors.py:24:101: E501 line too long (105 > 100 characters)


          Fix linting in monitors library

2203d30

abhishek-j16 changed the title ~~[build] Fix linting in monitors library~~ [unblock build] Fix linting in monitors library

Contributor Author

abhishek-j16 commented Sep 30, 2024 •

edited

Loading

@nzlosh - a quick one to unblock the new version release build. Ideally, the pre-commit checks should be able to catch linting issues. Thanks!

nzlosh merged commit 8ade8fd into StackStorm-Exchange:master

1 check passed

abhishek-j16 deleted the ajagannath/fixlint branch

October 1, 2024 12:01

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet