Closed iboates closed 2 months ago
Thanks for the PR @iboates and for the review @Zeitsperre!
Is it okay to merge now, or are you still modifying something?
Is it okay to merge now, or are you still modifying something?
Ok to merge
Hi @iboates & @Zeitsperre, I have invited you both for this repository maintainer. Please accept my invitation. Thank you very much for all of your efforts and contributions to this package. It is a lot more enhanced than before.
I have set up a Github action pipeline to run tests for every push & PR to master, as well as once a day.
I have disabled a lot of tests because they seem to be using other, less popular testing frameworks, or are dependent on local data.
I am interested in building out more tests to cover individual functionalities more comprehensively, and to use it as a springboard to make more comprehensive coverage of the geoserver API.
Do you have any objections to me removing or otherwise heavily modifying the existing tests?
by the way, I tried to write a pipeline to work on Windows as well, but I really had no idea what I was doing. I couldn't use docker containers, and I was making progress on installing things manually, but ultimately I decided it's not worth the effort. Maybe someone else can add Windows support in the future. My work is still there, just commented out.