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

Il 547 add metadata to overviews #899

Merged
merged 7 commits into from
Jun 13, 2024

Conversation

MerlinKallenbornAA
Copy link
Collaborator

Description

No description.

Before Merging

  • Review the code changes
    • Unused print / comments / TODOs
    • Missing docstrings for functions that should have them
    • Consistent variable names
    • ...
  • Update changelog.md if necessary
  • Commit messages should contain a semantic label and the ticket number
    • Consider squashing if this is not the case

@maxhammeralephalpha maxhammeralephalpha force-pushed the IL-547-add-metadata-to-overviews branch 2 times, most recently from bf736e6 to 236bf8b Compare June 11, 2024 15:26
@MerlinKallenbornAA MerlinKallenbornAA force-pushed the IL-547-add-metadata-to-overviews branch 2 times, most recently from b9e8aaa to e9cda34 Compare June 12, 2024 11:33
@MerlinKallenbornAA MerlinKallenbornAA marked this pull request as ready for review June 12, 2024 11:33
@MerlinKallenbornAA MerlinKallenbornAA force-pushed the IL-547-add-metadata-to-overviews branch from 7d06bdc to 4899fba Compare June 13, 2024 09:16
@Aleph-Alpha Aleph-Alpha deleted a comment from SebastianNiehusAA Jun 13, 2024
@MerlinKallenbornAA MerlinKallenbornAA merged commit 38998d7 into main Jun 13, 2024
4 checks passed
@MerlinKallenbornAA MerlinKallenbornAA deleted the IL-547-add-metadata-to-overviews branch June 13, 2024 11:43
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.

4 participants