Skip to content

IO module now behaves according to issue #121.#125

Open
NOhs wants to merge 3 commits intomasterfrom
improved_io_module
Open

IO module now behaves according to issue #121.#125
NOhs wants to merge 3 commits intomasterfrom
improved_io_module

Conversation

@NOhs
Copy link
Collaborator

@NOhs NOhs commented Oct 27, 2020

Apart from the changes in behaviour as described in issue #121, the new implementation

  • gets rid of a lot of Path <-> str back and forth conversions,
  • it removes duplicates by consistently using sets instead of manually cleaning up
  • it tries to more closely use terminology used throughout the project
  • it tries to split up the code into reasonable helper functions
  • it tries to improve the expressiveness of the "main" function

@NOhs NOhs force-pushed the improved_io_module branch from 20fca61 to 41e287f Compare October 27, 2020 23:20
@GPMueller GPMueller force-pushed the master branch 2 times, most recently from 1688aa5 to d663463 Compare October 30, 2020 12:37
@GPMueller
Copy link
Contributor

GPMueller commented Jul 6, 2021

Unfortunately some conflicts have appeared in 'clang_build/io_tools.py' in the meantime, which I'm not 100% sure how to fix correctly... Would be good to rebase this branch and check what the new CI says.

Edit: I think I merged correctly, but several tests are failing, including

  • test_hello_world_mwe
  • test_mwe_two_targets
  • test_automatic_include_folders

@GPMueller GPMueller force-pushed the improved_io_module branch from 86ca70d to af8cbc8 Compare July 6, 2021 09:41
@GPMueller GPMueller force-pushed the master branch 15 times, most recently from 5276c68 to 50e872b Compare July 27, 2021 20:57
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.

2 participants