Skip to content

Clean up neo4j.time public exports - #1191

Merged
robsdedude merged 2 commits into
neo4j:6.xfrom
robsdedude:remove-time-internals
May 28, 2025
Merged

Clean up neo4j.time public exports#1191
robsdedude merged 2 commits into
neo4j:6.xfrom
robsdedude:remove-time-internals

Conversation

@robsdedude

@robsdedude robsdedude commented May 20, 2025

Copy link
Copy Markdown
Member

Make undocumented internal constants, helper functions, and other items in neo4j.time private:

  • DATE_ISO_PATTERN
  • TIME_ISO_PATTERN
  • DURATION_ISO_PATTERN
  • NANO_SECONDS
  • AVERAGE_SECONDS_IN_MONTH
  • AVERAGE_SECONDS_IN_DAY
  • FORMAT_F_REPLACE
  • IS_LEAP_YEAR
  • DAYS_IN_YEAR
  • DAYS_IN_MONTH
  • round_half_to_even
  • symmetric_divmod
  • DateTimeType
  • DateType
  • TimeType
  • all other indirectly exposed items from imports (e.g. re as neo4j.time.re)

Deprecate ClockTime and its accessors

  • For each neo4j.time.Date, neo4j.time.DateTime, neo4j.time.Time
    • from_clock_time and to_clock_time methods
  • neo4j.time.ClockTime itself

@robsdedude
robsdedude force-pushed the remove-time-internals branch 2 times, most recently from 58e5b0d to 2f91ff8 Compare May 20, 2025 13:12
Make undocumented internal constants, helper functions, and other items in
`neo4j.time` private:

* `DATE_ISO_PATTERN`
* `TIME_ISO_PATTERN`
* `DURATION_ISO_PATTERN`
* `NANO_SECONDS`
* `AVERAGE_SECONDS_IN_MONTH`
* `AVERAGE_SECONDS_IN_DAY`
* `FORMAT_F_REPLACE`
* `IS_LEAP_YEAR`
* `DAYS_IN_YEAR`
* `DAYS_IN_MONTH`
* `round_half_to_even`
* `symmetric_divmod`
* `DateTimeType`
* `DateType`
* `TimeType`
* all other indirectly exposed items from imports (e.g. `re` as `neo4j.time.re`)
@robsdedude
robsdedude force-pushed the remove-time-internals branch from 2f91ff8 to 134e799 Compare May 20, 2025 15:29

@StephenCathcart StephenCathcart left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@robsdedude
robsdedude merged commit 26cb4ea into neo4j:6.x May 28, 2025
@robsdedude
robsdedude deleted the remove-time-internals branch May 28, 2025 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants