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

Change DLS datafile create time to datafileCreateTime #1664 #1668

Merged
merged 13 commits into from
Oct 1, 2024

Conversation

akintomiwa
Copy link

@akintomiwa akintomiwa commented Aug 9, 2024

Description

Switch out the column definition for createTime to datafileCreateTime.
Make the change for ISIS
update the type definition for Datafile

Testing instructions

Add a set up instructions describing how the reviewer should test the code

  • Review code
  • Check Actions build
  • Review changes to test coverage
  • {more steps here}

Agile board tracking

Closes #1664

@akintomiwa akintomiwa marked this pull request as draft August 9, 2024 15:50
Copy link

codecov bot commented Sep 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.35%. Comparing base (7ec33f3) to head (4e34c0c).
Report is 16 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1668   +/-   ##
========================================
  Coverage    95.35%   95.35%           
========================================
  Files          182      182           
  Lines         7844     7844           
  Branches      2482     2493   +11     
========================================
  Hits          7480     7480           
  Misses         329      329           
  Partials        35       35           
Flag Coverage Δ
common 94.17% <ø> (ø)
dataview 97.32% <100.00%> (ø)
download 95.99% <ø> (ø)
search 93.57% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

- also improve datafilePreview e2e tests
@louise-davies louise-davies marked this pull request as ready for review September 2, 2024 10:25
joelvdavies
joelvdavies previously approved these changes Sep 2, 2024
joshuadkitenge
joshuadkitenge previously approved these changes Sep 2, 2024
@louise-davies louise-davies merged commit 2633dcd into develop Oct 1, 2024
10 checks passed
@louise-davies louise-davies deleted the gfi-1664 branch October 1, 2024 08:28
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.

Change DLS datafile create time to datafileCreateTime
4 participants