Adrian Edwards
Adrian Edwards
i think @raisuraj2004 had a decent start on this in #3260 as well (although i havent fully reviewed it). maybe you can both collaborate on this for hacktoberfest?
You're welcome to take this on! Make sure you take a look at the previous linked attempts at this to help you (both as an outline of what you may...
Can you make the title of your pull request more descriptive of what is actually being fixed? Raw issue numbers by themselves arent very easy to read/understand at a glance
> fix: correct regex parsing of PostgreSQL connection string what I tried to do:- This update improves the parse_database_string function so it correctly pulls out the username, password, host, port,...
@Theminacious how much python experience do you have? This is probably somewhat more challenging as a first issue. I'd recommended starting with picking just one of the tasks to start...
Thanks for your contribution! In order to properly prioritize and review this, please make sure you are submitting your contributions with descriptive titles that explain what the fix is. While...
I also notice that this PR also contains the changes from https://github.com/chaoss/augur/pull/3463. If multiple sets of changes are included in a pull request, it makes the code a lot harder...
Augur is a data storage/collection tool. I think intentionally losing data, such as by adding truncation logic to the app, especially when we could just as easily increase the character...
Thanks for this fix! I am planning to address various database and schema jank very soon, however I'm not quite there yet. These kinds of changes tend to be breaking...
the CI is failing with this exception: ``` Traceback (most recent call last): augur-1 | File "/augur/.venv/bin/augur", line 10, in augur-1 | sys.exit(run()) augur-1 | ^^^^^ augur-1 | File "/augur/.venv/lib/python3.11/site-packages/click/core.py",...