Remove state from TemporalType

Open Beginner friendly
#529 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
72/100
Issue type
Refactor
Clarity
Mostly clear
Activity status
Quiet
Tech stack
java
Domain
tooling

Research direction

Locate TemporalType and the unit tests that roll back defaultExportFormat. Remove the state and the rollback logic, then make the type use its first format by default. Run the relevant unit tests in different orders to confirm they remain independent.

Written by the indexing model from the issue text.

Description

Follow-up of #481:

TemporalType has a field defaultExportFormat that keeps track of the last used format. The concept of property types is that they be stateless. Goal of this issue is to remove this field and to default to the first format instead.

Note that some unit tests perform a "rollback" of this field's value so the tests can be run in any order. This rollback should also be removed.

Dominant language
Java
Stars
44
Forks
21
Avg merge
2d 2h
Merged PRs (30d)
3

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from AuthMe/ConfigMe

All issues in AuthMe/ConfigMe

Similar issues

More Java issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.