Ben Straub

Results 336 comments of Ben Straub

@cicdguy any chance we can not have `.9000` appear as version? ![image](https://github.com/user-attachments/assets/45d68649-22cd-4fee-9d88-a5996a5cf459)

I really think copilot could handle this. We just have to write a really good prompt and review the checklist thoroughly that it makes.

@manciniedoardo We might want to break these up into individual issues? I think some of this environment handling tests are not easy - perhaps that is why there is no...

See this for more discussion on adding test coverage for `derive_vars_merged()` https://github.com/pharmaverse/admiral/pull/2619#issuecomment-2558241103

@jimrothstein since you are focused on #2636 I am going to free this issue up for someone else to look at.

Hi @1mgroot - we have released 1.3.0. Please start on this issue when you have time. It might be best to divide it up into smaller issues like @manciniedoardo did...

@MattBearham can you look into improving the test coverage for `derive_vars_tranposed` to 100% as potential first issue.

@MattBearham is taking derive_vars_dt_dtm_utils() and derive_vars_dt()