Skip to content

Comments

Rename ts, ep => timestamp, harvester#90

Merged
led02 merged 5 commits into26-implement-processing-stepfrom
84-rename-ts-ep-et-al-in-hermes-json-in-hermes-to-better-terms
Jan 11, 2023
Merged

Rename ts, ep => timestamp, harvester#90
led02 merged 5 commits into26-implement-processing-stepfrom
84-rename-ts-ep-et-al-in-hermes-json-in-hermes-to-better-terms

Conversation

@jkelling
Copy link
Contributor

Closes #84

@jkelling jkelling linked an issue Jan 11, 2023 that may be closed by this pull request
@jkelling jkelling changed the base branch from develop to 26-implement-processing-step January 11, 2023 11:59
@jkelling jkelling force-pushed the 84-rename-ts-ep-et-al-in-hermes-json-in-hermes-to-better-terms branch from 5ff1916 to 2556abe Compare January 11, 2023 12:00
@sdruskat
Copy link
Contributor

@jkelling I guess the tests should also be adapted, e.g. test/hermes_test/model/test_harvest_context.py which still includes the old names?

@jkelling jkelling force-pushed the 84-rename-ts-ep-et-al-in-hermes-json-in-hermes-to-better-terms branch from 2556abe to 509a063 Compare January 11, 2023 12:50
Co-authored-by: Stephan Druskat <sdruskat@users.noreply.github.com>
Copy link
Contributor

@sdruskat sdruskat left a comment

Choose a reason for hiding this comment

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

Good to go with the respective comments.

@led02 led02 merged commit e6b93e7 into 26-implement-processing-step Jan 11, 2023
@sdruskat sdruskat deleted the 84-rename-ts-ep-et-al-in-hermes-json-in-hermes-to-better-terms branch January 12, 2023 12:51
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.

Rename "ts", "ep" et al. in HERMES JSON in .hermes to better terms

3 participants