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.

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

MB-58033: Support for custom datetime layouts #1866

Merged
merged 9 commits into from
Sep 13, 2023
Merged

Conversation

CascadingRadium
Copy link
Member

@CascadingRadium CascadingRadium commented Aug 9, 2023

Jira

MB-58033

Description

  • Add 'percentstyle' date time layout format that mimics 'strftime' used for date time layout specification in C and Python.
  • Add 'isostyle' date time layout format that attempts to mimic 'DateTimeFormatter' which is used to specify date time layouts in Java.
  • Added unit tests

analysis/datetime/percent/percent.go Outdated Show resolved Hide resolved
Copy link
Member

@abhinavdangeti abhinavdangeti left a comment

Choose a reason for hiding this comment

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

Can you quickly set up a separate document highlighting all the support this will be adding, which we can later share with the docs team.

analysis/datetime/percent/percent.go Outdated Show resolved Hide resolved
abhinavdangeti
abhinavdangeti previously approved these changes Sep 7, 2023
Copy link
Member

@abhinavdangeti abhinavdangeti left a comment

Choose a reason for hiding this comment

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

Please wait for one more review here.

Thejas-bhat
Thejas-bhat previously approved these changes Sep 13, 2023
@abhinavdangeti
Copy link
Member

abhinavdangeti commented Sep 13, 2023

Also, not sure how you rebased this, but it seems you've cherry-picked a bunch of commits that have already been merged into this PR. Can you rebase over master and simply cherry-pick your commits and force push here.

@abhinavdangeti
Copy link
Member

@CascadingRadium I force-pushed all your commits over rebased master to avoid unnecessary cherry-picks of everything in between. Should you need to push more code here, make sure you start from a clean local branch checking-out the latest of origin/MB-58033 here.

@abhinavdangeti abhinavdangeti merged commit 1788a80 into master Sep 13, 2023
9 checks passed
@abhinavdangeti abhinavdangeti deleted the MB-58033 branch September 13, 2023 21:33
Thejas-bhat pushed a commit that referenced this pull request Oct 12, 2023
## Jira

[MB-58033](https://issues.couchbase.com/browse/MB-58033)

## Description
- Add 'percentstyle' date time layout format that mimics 'strftime' used
for date time layout specification in C and Python.
- Add 'isostyle' date time layout format that attempts to mimic
'DateTimeFormatter' which is used to specify date time layouts in Java.
- Added unit tests

---------

Co-authored-by: Abhinav Dangeti <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants