Skip to content

fix test_add_and_remove_module_path by replacing string comparison of paths by checking whether they point to the same path (since symlinks may cause trouble)#4312

Merged
ocaisa merged 1 commit intoeasybuilders:developfrom
boegel:fix_test_add_and_remove_module_path
Aug 7, 2023
Merged

fix test_add_and_remove_module_path by replacing string comparison of paths by checking whether they point to the same path (since symlinks may cause trouble)#4312
ocaisa merged 1 commit intoeasybuilders:developfrom
boegel:fix_test_add_and_remove_module_path

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Aug 6, 2023

No description provided.

… paths by checking whether they point to the same path (since symlinks may cause trouble)
@boegel boegel added this to the next release (4.8.1?) milestone Aug 6, 2023
@boegel boegel requested a review from ocaisa August 6, 2023 19:17
Copy link
Copy Markdown
Member

@ocaisa ocaisa left a comment

Choose a reason for hiding this comment

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

LGTM

@ocaisa ocaisa merged commit b8c153d into easybuilders:develop Aug 7, 2023
@boegel boegel deleted the fix_test_add_and_remove_module_path branch August 7, 2023 19:50
@boegel boegel changed the title fix test_add_and_remove_module_path by replacing string comparison onf paths by checking whether they point to the same path (since symlinks may cause trouble) fix test_add_and_remove_module_path by replacing string comparison of paths by checking whether they point to the same path (since symlinks may cause trouble) Aug 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants