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

Deprecate the schema builder tool #947

Closed
n3101 opened this issue Oct 27, 2021 · 3 comments · Fixed by #950
Closed

Deprecate the schema builder tool #947

n3101 opened this issue Oct 27, 2021 · 3 comments · Fixed by #950

Comments

@n3101
Copy link

n3101 commented Oct 27, 2021

The schema builder tool appears not to be used by anyone (no responses received to question in #930) and has not received any development effort in years.

While a schema builder is in principle a good idea, this implementation isn't the way forward. Deprecate this now for removal in v2.0.

@n3101 n3101 added this to the Backlog milestone Oct 27, 2021
@t92549
Copy link
Contributor

t92549 commented Nov 3, 2021

I have tried to use it to see if it even works anymore and as far as I can see it doesn't sadly

@n3101
Copy link
Author

n3101 commented Nov 9, 2021

@t92549 So no loss then. Kill!

@n3101
Copy link
Author

n3101 commented Nov 16, 2021

Following a team discussion, it appears that the correct way forward for removing an entire package is to amend its Readme to say that it will be removed in Gaffer 2.0, so please do this.

t92549 added a commit that referenced this issue Nov 17, 2021
t92549 added a commit that referenced this issue Nov 24, 2021
* Add deprecation notice to schema builder README

* gh-947: Fix rewording
@t92549 t92549 modified the milestones: v1_backlog, v1.21.0 Nov 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants