Add tech stack for java

red-gate / Tech-Radar

A Tech Radar for Redgate

21 stars 14 forks source link

Add tech stack for java #350

Closed Barry-RG closed 2 years ago

Barry-RG commented 3 years ago

[x] Are you happy for the content of this change to be publicly visible?
[x] Are all new or updated entries marked as isNew = true?
[x] Does radar.csv render correctly when viewed on Github?
[x] Does the updated tech radar display as you expect?

fffej commented 3 years ago

@Barry-RG Have you got the right line endings settings? It's difficult to use the diff at the moment and I think there's a typo in one of the categories so it doesn't display correctly.

Barry-RG commented 3 years ago

@fffej sorry, didn't realize what was causing that. Excel stripped the quotes out. Thought it was just github being strange

fffej commented 2 years ago

@Barry-RG We made some changes to the tech radar a year or so back when we removed libraries from the radar; instead trying to focus on standardizing the "glue".

I guess JDBC is an important one to call out (we should be consistent across products on that), but JANSI, GSON and JUnit feel like something a team could flex on without affecting anyone else? If you agree, could you remove them?

Barry-RG commented 2 years ago

@fffej JANSI and GSON that may be fair. However, I would stick with having JUnit there as its precedent there for it to be included as part of a Java stack as its a usually good to have a test framework specified.

fffej commented 2 years ago

@Barry-RG Sounds good - if you could merge the conflicts and drop JANSI/GSON then I'll press merge 😄