Ben Straub

Results 336 comments of Ben Straub

Maybe we can start to leverage pharmaverse resources to get v2 out the door and make additional enhancements if needed?

@manciniedoardo @bundfussr @ddsjoberg I think this is it - you guys see anything amiss?

> Shouldn't we have "[skip vbump]" in the title to avoid that the version is changed to 1.1.0.9000? I placed this in the Release Task List as well ![image](https://github.com/pharmaverse/admiral/assets/10111024/f21734a3-ecc7-4f3f-be5c-5af7c47e9b97)

Hi @imccay13 I see this is draft, but just wanted to say there seems to be quite a few files that are not needed for this update in this PR.

Hi @imccay13 if they pass all the checks (you can ignore link one), then please click ready for review. For the new function, I'm just glancing at this PR so...

ooohhh I was just having a thought about dirty data - should a simple message be returned to the user if nothing is flagged. I'm assuming the user would be...

Hi @imccay13 - we have a release planned for June 2nd. It would be really nice to finish this PR by EOW please. Please implement our requested changes so we...

@imccay13 I created the Discussion as an issue and updated the title of the PR. This way once the PR merges into main it will automatically close the issue. The...

@imccay13 can you make these last couple of tweaks and then we can merge in!! Also, please add yourself to description file or ping me in TEAMS and I can...

I added you to the repo @tuiaj - thanks for doing the dummy PR. Do you have any questions?