Skip to content

[ENG-10665] support osfmetrics migration#95

Merged
aaxelb merged 8 commits into
CenterForOpenScience:mainfrom
aaxelb:10665-support-osfmetrics-migration
Apr 6, 2026
Merged

[ENG-10665] support osfmetrics migration#95
aaxelb merged 8 commits into
CenterForOpenScience:mainfrom
aaxelb:10665-support-osfmetrics-migration

Conversation

@aaxelb

@aaxelb aaxelb commented Mar 31, 2026

Copy link
Copy Markdown
Collaborator
  • improve test setup/teardown, isolating test runs with index-name prefixes
  • index CyclicRecord by cycle_coverage, not by time created
  • more consistent date formatting in index names (use semver-like 2026.3.31 to align with the timeseries_timeparts field)
  • move djelme_registry.all_recordtypes to semi-private attr (require interacting with registry thru methods, not direct access to mutable dict)
  • improve docs

@aaxelb aaxelb marked this pull request as ready for review March 31, 2026 15:17
@aaxelb aaxelb requested a review from felliott April 3, 2026 12:59

@felliott felliott left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looks good, no notes!

@aaxelb aaxelb merged commit 1b644bb into CenterForOpenScience:main Apr 6, 2026
16 checks passed
@aaxelb aaxelb deleted the 10665-support-osfmetrics-migration branch April 6, 2026 13:39
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