Summitting Pull Request #916
Replies: 2 comments
-
after I submitted the ticket, i realized you might be interested summary of the new features I developed, I don't really have a inclusive list but i will link to a few of the relevant decussations. It will take me a while to separate the stuff in to PR but i am willing to if these thing are useful.
A quick note: Python is my go-to language, though I’ve reverse-engineered several projects in this codebase’s languages. I might be missing some conventions or steps, so please don’t hesitate to point out anything I’m doing wrong—I won’t be insulted! |
Beta Was this translation helpful? Give feedback.
-
Make sure you've run |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
I’ve been working on several fixes locally but didn’t commit them until I reviewed the discussion thread. After pulling the latest
main
and preparing my PR, thetask pr
checks failed and unrelated files were modified. It looks like the root cause lies in the currentmain
branch. I’m going to create a feature branch with what it is changing so you can review them. In the meantime, could you let me know if there’s a different setup or task I should’ve run before doing a pull request?Beta Was this translation helpful? Give feedback.
All reactions