Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

api: unify usage of external types #306

Merged
merged 2 commits into from
Jun 30, 2023
Merged

api: unify usage of external types #306

merged 2 commits into from
Jun 30, 2023

Conversation

oleg-jukovec
Copy link
Collaborator

After the patch Decimal, Datetime and uuid used in the same way: by a value.

I didn't forget about (remove if it is not applicable):

Related issues:

Part of #238

DifferentialOrange
DifferentialOrange reviewed Jun 23, 2023
View reviewed changes
README.md Outdated Show resolved Hide resolved
@oleg-jukovec oleg-jukovec force-pushed the oleg-jukovec/gh-238-immutable branch from 8a732b2 to 437dca9 Compare June 23, 2023 10:55
@oleg-jukovec oleg-jukovec force-pushed the oleg-jukovec/gh-238-immutable branch from 437dca9 to 22de648 Compare June 24, 2023 17:45
@oleg-jukovec
api: the Decimal type is immutable
63b1024 
The patch forces the use of objects of type Decimal instead
of pointers.

Part of #238
@oleg-jukovec
api: the Datetime type is immutable
bfcf5e6 
The patch forces the use of objects of type Datetime instead
of pointers.

Part of #238
@oleg-jukovec oleg-jukovec force-pushed the oleg-jukovec/gh-238-immutable branch from 22de648 to bfcf5e6 Compare June 30, 2023 17:41
@oleg-jukovec oleg-jukovec merged commit b30e2b6 into master Jun 30, 2023
@oleg-jukovec oleg-jukovec deleted the oleg-jukovec/gh-238-immutable branch June 30, 2023 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DifferentialOrange DifferentialOrange DifferentialOrange approved these changes

@GRISHNOV GRISHNOV GRISHNOV approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@oleg-jukovec @DifferentialOrange @GRISHNOV