Skip to content
This repository has been archived by the owner on Apr 8, 2023. It is now read-only.

Review and merge all upstream's pull requests #3

Closed
6 of 9 tasks
omissis opened this issue Sep 21, 2022 · 18 comments
Closed
6 of 9 tasks

Review and merge all upstream's pull requests #3

omissis opened this issue Sep 21, 2022 · 18 comments
Assignees
Milestone

Comments

@omissis omissis self-assigned this Sep 21, 2022
@omissis omissis added this to the v0.10.0 milestone Sep 21, 2022
@scop
Copy link

scop commented Oct 3, 2022

Forward ported here as omissis/go-jsonschema#17

@atombender
Copy link
Contributor

Hi, I see you started this fork. Want to take over as maintainer? I'm not using go-jsonschema these days and don't really have time to maintain it.

@omissis
Copy link
Owner Author

omissis commented Oct 3, 2022

Hey @atombender , I'd be glad to!

@atombender
Copy link
Contributor

@omissis Sorry I've been late to address this. Still interested? Do you have a handle on other forks that have made changes you might want to pull in?

@omissis
Copy link
Owner Author

omissis commented Feb 22, 2023

Hey @atombender, I haven't looked at new PRs this year so the status is still the one describe above in the issue description, but I will do one more round soon to ensure everything of interest has been included. Besides that yes, I am still interested. I will probably explore subschemas soon, and possibly look into validation later on this year: we're actively using the binary within our kubernetes distribution (https://github.com/sighupio/fury-distribution/blob/feature/furyctl-next/Makefile#L4) so it would be cool to get the chance to maintain it. any preference on how to proceed?

@atombender
Copy link
Contributor

Cool! I could transfer my repo to you, but you already have a fork, and I don't think GitHub has a way to make another fork the "main" repo? Otherwise I could just put a link in the readme and archive my repo.

@shanduur
Copy link

Great to see a work done on a fork! I think the archive and add a URL to @omissis fork is best way of handling this.

@omissis
Copy link
Owner Author

omissis commented Feb 27, 2023

I think transferring the repository could be a bit better in terms of handling redirects, but I am not 100% sure that would suffice in order to keep the go module name unchanged and avoid disrupting downstream users of the library: I need to do a bit of research on the topic before deciding if it's a good idea.
Otherwise we can simply point to my repository from atombender's and have the users update their references: that would work for me, too.

@shanduur
Copy link

shanduur commented Mar 9, 2023

@omissis @atombender any updates? If help is needed, I'm here 😄

@omissis
Copy link
Owner Author

omissis commented Mar 9, 2023

hey 👋 no updates, I just need to look into the matter :)

@omissis
Copy link
Owner Author

omissis commented Mar 25, 2023

hi @atombender! I finally took the time to run some tests to ensure the transfer won't cause disruption to downstream users and I would go with repository transfer. I already renamed this one to allow the move, so I am ready when you are :)

@omissis
Copy link
Owner Author

omissis commented Mar 25, 2023

For future reference: github repository transfer and go mod do handle everything transparently, so the users will keep receiving updates when referencing github.com/atombender/go-jsonschema even if the changes happen here. I might be changing the module name in a future release to avoid confusion, but I'll plan that further down the road.

@atombender
Copy link
Contributor

Great! I tried transferring, but GitHub fails with:

omissis already has a repository in the atombender/go-jsonschema network

I wonder if it's possible for you to "unfork" the repo somehow?

@omissis
Copy link
Owner Author

omissis commented Mar 31, 2023

ok @atombender I sent a support request to detach the fork, hopefully it's going to be processed soon enough

@omissis
Copy link
Owner Author

omissis commented Mar 31, 2023

hi @atombender they detached this repo, can you try again? thanks! 🙏

@atombender
Copy link
Contributor

Done!

@omissis
Copy link
Owner Author

omissis commented Mar 31, 2023

Awesome! 🚀 Thank you for taking the time and for having put in the work to start this repo! ❤️

@omissis
Copy link
Owner Author

omissis commented Apr 8, 2023

closing this is issue, as the conversation moved to the original repo: omissis/go-jsonschema#53

@omissis omissis closed this as completed Apr 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants