search

sociomantic-tsunami / ocean

General purpose, platform-dependent, high-performance library for D
Other
61 stars 56 forks source link

Remove v2.071.2.s* from the test matrix #829

Closed Geod24 closed 4 years ago

Geod24 commented 4 years ago 
This compiler is long gone and there is no interest in supporting it in the future.
This is the lowest hanging fruit in the matrix and will allow upstream to move
forward immediately.

CC @joseph-wakeling-frequenz

codecov[bot] commented 4 years ago

Codecov Report

Merging #829 into v5.x.x will increase coverage by 0.08%. The diff coverage is n/a.

codecov[bot] commented 4 years ago

Codecov Report

Merging #829 into v5.x.x will increase coverage by 0.08%. The diff coverage is n/a.

codecov[bot] commented 4 years ago

Codecov Report

Merging #829 into v5.x.x will increase coverage by 0.08%. The diff coverage is n/a.

ben-palmer-sociomantic commented 4 years ago

LGTM

joseph-wakeling-frequenz commented 4 years ago

Just to check, are there any downstream apps/libs in tsunami still using (relying on?) this compiler?

Geod24 commented 4 years ago

Yes, the nodes, but they will be updated in due time.

joseph-wakeling-frequenz commented 4 years ago

Yea, fair enough. We can tweak their requirements once they update their dependencies.

joseph-wakeling-frequenz commented 4 years ago

(As a general rule I prefer to operate the other way round -- drop the downstream requirement first, then drop support upstream -- but in this case I think we can be practical:-)