New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fix flake8 error in publisher example #229

Merged

JafarAbdi merged 1 commit into main from one-line-to-rule-them-all

Jan 6, 2025

Collaborator

sea-bass commented Jan 6, 2025 •

edited

Loading

CI (on Rolling only) was showing this failure:

build/generate_parameter_module_example/pytest.xml: 3 tests, 0 errors, 1 failure, 1 skipped
- generate_parameter_module_example.test.test_flake8 test_flake8
  <<< failure message
    AssertionError: Found 1 code style errors / warnings:
      ./generate_parameter_module_example/minimal_publisher.py:39:1: CNL100 Class definition does not have a new line.
    assert 1 == 0
  >>>


          Fix flake8 error in publisher example

f3784c8

sea-bass requested review from christophfroehlich and pac48

January 6, 2025 14:44

JafarAbdi approved these changes

View reviewed changes

JafarAbdi merged commit ee8a1f3 into main

7 checks passed

JafarAbdi deleted the one-line-to-rule-them-all branch

January 6, 2025 14:51

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet