Closed TomHAnderson closed 1 year ago
@TomHAnderson Ah, apologies. First time contributing. I've fixed the 3 lines flagged by the codesniffer check. What's the right way to push that through?
@dkmuir The CS isn't your problem. I've opened #73 to fix the underlying issue and will merge that first then rebase your commit.
@dkmuir To answer your question, a PR to my PR'd branch would be the way to do it.
Merged #73, would you mind rebasing?
Moved from #70 onto a different branch
@dkmuir