tegola icon indicating copy to clipboard operation
tegola copied to clipboard
verified publisher icon go-spatial

provider/postgis: Clean out deprecated connection params

Open ARolek opened this issue 1 year ago • 4 comments

In v0.15.0 support for a connection string uri was added to provider/postgis. When this change was made, we marked the old connection params as deprecated. These params now need to be removed. This issue is for tracking this removal from the following places:

  • [ ] provider/postgis: This package needs to remove support for the old connection params. The README should also be updated.
  • [ ] .github/: The CI uses the old connection params in a few places and needs to be migrated.

ARolek avatar Feb 27 '24 21:02 ARolek

I can take it. Do you think it makes sense to create Github project to collect open work like this? Have to admit I completely forgot about it - and actually volunteered for it back then. :disappointed:

iwpnd avatar Feb 28 '24 12:02 iwpnd

I can take it.

That would be awesome! Your recent PR made me realize this hasn't been done but should absolutely be cleaned up. I'm on a "purge all the unnecessary things" kick right now 😉

Do you think it makes sense to create Github project to collect open work like this

I would absolutely be in favor of this. We could then organize issues based on priority if contributors are looking to pick up an issue. I can spin up a project called "Roadmap" and we can start adding issues to it. How does that sound?

ARolek avatar Feb 28 '24 16:02 ARolek

Great 👌🏻 let’s do that

iwpnd avatar Feb 28 '24 17:02 iwpnd

@iwpnd done. I also invited you to it so you can help groom the backlog ;-)

ARolek avatar Feb 28 '24 21:02 ARolek

@iwpnd have you had a chance to look at this issue? I'm hoping to include these changes in the upcoming release. The upcoming release has a lot of vuln squashing in it, so I would like to get a version cut in the not too distant future. LMK if you want to tag team this one.

ARolek avatar Mar 18 '24 02:03 ARolek

Gosh 19d ago already. I'm swamped at work right now and happy to not touch the keyboard in the evening. But I do have a three hour train ride on Wednesday reserved for it. ☝🏻🤞🏻

iwpnd avatar Mar 18 '24 06:03 iwpnd

@iwpnd No worries!!

I'm swamped at work right now and happy to not touch the keyboard in the evening.

I can relate. A lot to juggle right now. No pressure.

But I do have a three hour train ride on Wednesday reserved for it.

Excellent. Feel free to push up a branch if you need some help along the way. Thanks for tackling this one!

ARolek avatar Mar 18 '24 15:03 ARolek