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

Separate out @deephaven/iris-grid package #28

Merged
merged 2 commits into from
May 19, 2021

Conversation

mofojed
Copy link
Member

@mofojed mofojed commented May 17, 2021

  • @deephaven/iris-grid package now has IrisGrid, IrisGridUtils, etc
  • @deephaven/mocks package contains common mocks
  • Change @deephaven/log to read in log level from DEEPHAVEN_LOG_LEVEL. Also made it default to SILENT mode when in 'test' mode, and DEBUG2 when in 'development' mode.

Merge #27 first

@mofojed mofojed requested a review from vbabich May 17, 2021 21:04
@mofojed
Copy link
Member Author

mofojed commented May 17, 2021

Precursor to #23

@mofojed mofojed changed the title mofojed/issue23 iris grid Separate out @deephaven/iris-grid package May 17, 2021
vbabich
vbabich previously approved these changes May 18, 2021
mofojed added 2 commits May 19, 2021 10:12
- @deephaven/iris-grid package now has IrisGrid, IrisGridUtils, etc
- @deephaven/mocks package contains common mocks
- Change @deephaven/log to read in log level from DEEPHAVEN_LOG_LEVEL. Also made it default to SILENT mode when in 'test' mode, and DEBUG2 when in 'development' mode.
@mofojed mofojed force-pushed the mofojed/issue23-iris-grid branch from ccae69b to c7385a9 Compare May 19, 2021 14:19
@mofojed mofojed requested a review from vbabich May 19, 2021 14:20
@mofojed mofojed self-assigned this May 19, 2021
@mofojed mofojed merged commit 349fa54 into deephaven:main May 19, 2021
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