Skip to content

Code refactors to the ament_index_python test#43

Merged
Blast545 merged 6 commits intomasterfrom
blast545/py_refactor
Jan 27, 2020
Merged

Code refactors to the ament_index_python test#43
Blast545 merged 6 commits intomasterfrom
blast545/py_refactor

Conversation

@Blast545
Copy link
Copy Markdown
Contributor

The refactors are submitted in different commits for easier tracking, these are:

  • Replace try/except to catch exceptions, using the with context clause now
  • pathlib preferred over os.path functions
  • Replace format with f-strings
  • Removing comments

Signed-off-by: Jorge J. Perez <jjperez@ekumenlabs.com>
Signed-off-by: Jorge J. Perez <jjperez@ekumenlabs.com>
Signed-off-by: Jorge J. Perez <jjperez@ekumenlabs.com>
Signed-off-by: Jorge J. Perez <jjperez@ekumenlabs.com>
@Blast545 Blast545 requested a review from ivanpauno January 24, 2020 13:54
@ivanpauno ivanpauno changed the title Code refactors to the ament_index_python package Code refactors to the ament_index_python test Jan 24, 2020
@ivanpauno ivanpauno added the in review Waiting for review (Kanban column) label Jan 24, 2020
This reverts commit 50f7a7b.

Signed-off-by: Jorge J. Perez <jjperez@ekumenlabs.com>
@Blast545 Blast545 force-pushed the blast545/py_refactor branch from f614414 to e9bf2cb Compare January 24, 2020 16:24
@Blast545 Blast545 requested a review from ivanpauno January 24, 2020 16:26
Copy link
Copy Markdown
Contributor

@ivanpauno ivanpauno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with green CI!

Signed-off-by: Jorge J. Perez <jjperez@ekumenlabs.com>
@Blast545 Blast545 requested a review from dirk-thomas January 24, 2020 19:18
@Blast545
Copy link
Copy Markdown
Contributor Author

Blast545 commented Jan 24, 2020

CI:

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@Blast545 Blast545 merged commit ffa268a into master Jan 27, 2020
@delete-merged-branch delete-merged-branch bot deleted the blast545/py_refactor branch January 27, 2020 15:18
@Blast545 Blast545 removed the in review Waiting for review (Kanban column) label Jan 27, 2020
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.

3 participants