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

Force using clock package everywhere #1665

Open
NikitaSkrynnik opened this issue Sep 17, 2024 · 0 comments
Open

Force using clock package everywhere #1665

NikitaSkrynnik opened this issue Sep 17, 2024 · 0 comments
Assignees
@Ex4amp1e

Comments

@NikitaSkrynnik
Copy link
Contributor

NikitaSkrynnik commented Sep 17, 2024 •
edited
Loading

Main Issue

#1666

Description

clock package should be used in all chain elements instead of time as it's useful for writing unit tests.

Solution

Add a linter checking that time package is used only inside clock package
@NikitaSkrynnik NikitaSkrynnik self-assigned this Sep 17, 2024
@NikitaSkrynnik NikitaSkrynnik mentioned this issue Sep 17, 2024
Open
@NikitaSkrynnik NikitaSkrynnik changed the title Get rid of clock package Force using clock package everywhere Sep 19, 2024
@NikitaSkrynnik NikitaSkrynnik moved this to In Progress in Release v1.14.0 Sep 20, 2024
@denis-tingaikin denis-tingaikin moved this from In Progress to Moved to next release in Release v1.14.0 Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Ex4amp1e Ex4amp1e

Labels
None yet
Projects
Release v1.14.0
Status: Moved to next release
Release v1.14.1
Status: No status
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Ex4amp1e @NikitaSkrynnik