Skip to content

Conversation

@j-atkins
Copy link
Collaborator

@j-atkins j-atkins commented Feb 5, 2026

This PR adds a step to generate a human-readable record of all problems which occurred across the expedition (including their full message), which could be useful to refer back to when students are making their presentations.

Also tidies up some of the problems_encountered directory strucutre.

Copy link
Member

@erikvansebille erikvansebille left a comment

Choose a reason for hiding this comment

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

Code changes look good to me; but could you also add some unit-tests so that we don’t unintentionally break this functionality later?

@j-atkins
Copy link
Collaborator Author

j-atkins commented Feb 6, 2026

Code changes look good to me; but could you also add some unit-tests so that we don’t unintentionally break this functionality later?

I've added more testing now 👍

@j-atkins j-atkins merged commit 0317127 into problems Feb 6, 2026
9 of 10 checks passed
@j-atkins j-atkins deleted the problems-report branch February 6, 2026 14: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.

2 participants