Code

Zoned date time#3

Code

Open

jejking-tw wants to merge 11 commits intoThoughtWorksInc:option-0-no-apiThoughtWorksInc/developer-joyofenergy-java:option-0-no-apifrom jejking-tw:zoned-date-timejejking-tw/developer-joyofenergy-java-new:zoned-date-timeCopy head branch name to clipboard

+295-181

Commits

Commits on Jul 15, 2024

• WIP - assorted improvements to readability. Introduces ZonedDateTime

  jejking-tw

  committed
  23e33a7

• WIP - changes readings generator to use a stream, more use of zoned date time

  jejking-tw

  committed
  89e4c02

Commits on Jul 16, 2024

• Completes chunk of proposed refactoring.

  Show description for 4c379c1
  jejking-tw

  committed
  4c379c1

Commits on Jul 21, 2024

• Update src/main/java/tw/joi/energy/config/ElectricityReadingsGenerator.java

  Show description for d2eb50d
  

  jejking-tw

  and

  jeremyhuiskamp

  authored
  d2eb50d

• Update src/test/java/tw/joi/energy/config/ElectricityReadingsGeneratorTest.java

  Show description for 19e10a6
  

  jejking-tw

  and

  jeremyhuiskamp

  authored
  19e10a6

• Changes PricePlanTest to use isEqualByComparingTo on BigDecimal

  jejking-tw

  committed
  c9c2268

• Makes it clearer that the fixed time stamp is just arbitary

  jejking-tw

  committed
  54a628c

• Makes it clear that the fixed clock is arbitary.

  jejking-tw

  committed
  20ce194

• Provides an additional constructor for PricePlan that initiates an immutable empty map of peak time modifiers. Modifies the existing constructor to remove unnecessary (and broken) conversion from…

  Show description for 6baa03c
  jejking-tw

  committed
  6baa03c

• applies code formatting rules

  jejking-tw

  committed
  50af408

• rationalises test method names to align with @DisplayName annotation

  jejking-tw

  committed
  aa0836c