-
Notifications
You must be signed in to change notification settings - Fork 105
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
Changes to make ament_flake8 work with v6+ #459
Conversation
Signed-off-by: Michael Carroll <mjcarroll@intrinsic.ai>
Signed-off-by: Michael Carroll <mjcarroll@intrinsic.ai>
Hi @mjcarroll, I hope the dependency of |
@tylerjw To take a look as well. |
Signed-off-by: Chris Lalancette <clalancette@gmail.com>
@clalancette Thanks for implementing these changes. Are we planning to add this change to other versions? I'm most concerned about the Humble version. |
@Mergifyio backport iron humble |
✅ Backports have been created
|
* Changes to make ament_flake8 work with v6+ * Correct make_file_checker_manager call * Fix compatibility with flake8 5. Signed-off-by: Michael Carroll <mjcarroll@intrinsic.ai> Signed-off-by: Chris Lalancette <clalancette@gmail.com> (cherry picked from commit fc02432) # Conflicts: # ament_flake8/ament_flake8/main.py
* Changes to make ament_flake8 work with v6+ * Correct make_file_checker_manager call * Fix compatibility with flake8 5. Signed-off-by: Michael Carroll <mjcarroll@intrinsic.ai> Signed-off-by: Chris Lalancette <clalancette@gmail.com> (cherry picked from commit fc02432)
* Changes to make ament_flake8 work with v6+ * Correct make_file_checker_manager call * Fix compatibility with flake8 5. Signed-off-by: Michael Carroll <mjcarroll@intrinsic.ai> Signed-off-by: Chris Lalancette <clalancette@gmail.com> (cherry picked from commit fc02432) Co-authored-by: Michael Carroll <mjcarroll@intrinsic.ai>
@Mergifyio backport iron |
✅ Backports have been created
|
* Changes to make ament_flake8 work with v6+ * Correct make_file_checker_manager call * Fix compatibility with flake8 5. Signed-off-by: Michael Carroll <mjcarroll@intrinsic.ai> Signed-off-by: Chris Lalancette <clalancette@gmail.com> (cherry picked from commit fc02432)
* Changes to make ament_flake8 work with v6+ * Correct make_file_checker_manager call * Fix compatibility with flake8 5. Signed-off-by: Michael Carroll <mjcarroll@intrinsic.ai> Signed-off-by: Chris Lalancette <clalancette@gmail.com> (cherry picked from commit fc02432) Co-authored-by: Michael Carroll <mjcarroll@intrinsic.ai>
* Changes to make ament_flake8 work with v6+ * Correct make_file_checker_manager call * Fix compatibility with flake8 5. Signed-off-by: Michael Carroll <mjcarroll@intrinsic.ai> Signed-off-by: Chris Lalancette <clalancette@gmail.com> Signed-off-by: WU MENGHUNG <wu.menghung@jp.panasonic.com>
parse_preliminary_options_and_args
andparse_preliminary_options
have both been removed in favor ofparse_args
in this commit: PyCQA/flake8@0d667a7(No corresponding PR as far as I can tell).
This is the minimal set of changes to get build/test passing on flake8 v6