Skip to content
This repository has been archived by the owner on Aug 19, 2020. It is now read-only.

Remove support for discontinued json project schema snapshots #1195

Merged
merged 8 commits into from
Oct 18, 2018

Conversation

eskatos
Copy link
Member

@eskatos eskatos commented Oct 18, 2018

Warn and suggest to delete the now unnecessary file.

Signed-off-by: Paul Merlin <paul@gradle.com>
Signed-off-by: Paul Merlin <paul@gradle.com>
Signed-off-by: Paul Merlin <paul@gradle.com>
@eskatos eskatos added this to the 1.0-RC13 milestone Oct 18, 2018
@eskatos eskatos self-assigned this Oct 18, 2018
@eskatos eskatos requested a review from bamboo October 18, 2018 16:32
@eskatos eskatos changed the title Remove support for discontinued json project schema snapthots Remove support for discontinued json project schema snapshots Oct 18, 2018
@eskatos eskatos changed the title Remove support for discontinued json project schema snapshots WIP Remove support for discontinued json project schema snapshots Oct 18, 2018
@eskatos eskatos changed the title WIP Remove support for discontinued json project schema snapshots Remove support for discontinued json project schema snapshots Oct 18, 2018
@eskatos eskatos changed the title Remove support for discontinued json project schema snapshots WIP Remove support for discontinued json project schema snapshots Oct 18, 2018
by moving json support into that test as a first step
this coverage should be turned into a quicker test at some point

Signed-off-by: Paul Merlin <paul@gradle.com>
Signed-off-by: Paul Merlin <paul@gradle.com>
@eskatos eskatos changed the title WIP Remove support for discontinued json project schema snapshots Remove support for discontinued json project schema snapshots Oct 18, 2018
The `ProjectSchema` is an implementation detail that will have to go
through some changes as we optimise the generation of accessors and
this test brings unwanted inertia for very little in return as we
already have exhaustive coverage of all the accessors.
@bamboo bamboo merged commit 95ae6d8 into develop Oct 18, 2018
@eskatos eskatos deleted the eskatos/json-schema/remove branch October 19, 2018 06:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants