Skip to content

docs: update readme instructions for running the collection-task locally - #54

Merged
tombrooks248 merged 2 commits into
mainfrom
docs/add-run-collection-task-locally-instructions-to-readme
May 13, 2026
Merged

docs: update readme instructions for running the collection-task locally#54
tombrooks248 merged 2 commits into
mainfrom
docs/add-run-collection-task-locally-instructions-to-readme

Conversation

@tombrooks248

@tombrooks248 tombrooks248 commented May 12, 2026

Copy link
Copy Markdown
Contributor

Description

Updating the collection-task README.md to make running collection-task easier to follow including a few tips about how and test any code changes which might be made in collection-task's sister repositories.

Add a run:: to the Makefile so that make run it actually runs the pipelines.

Change make clean:: to clean:: just so its fits to the code is more consistent.

Related Tickets & Documents

QA Instructions, Screenshots, Recordings

Please replace this line with instructions on how to test your changes, a note
on the devices and browsers this has been tested on, as well as any relevant
images for UI changes.

Added/updated tests?

We encourage you to keep the code coverage percentage at 80% and above. Please refer to the Digital Land Testing Guidance for more information.

  • Yes
  • No, and this is why: Just docs, no code so no tests required.
  • I need help with writing tests

[optional] Are there any post deployment tasks we need to perform?

[optional] Are there any dependencies on other PRs or Work?

@tombrooks248
tombrooks248 force-pushed the docs/add-run-collection-task-locally-instructions-to-readme branch from f83c842 to 4119eb1 Compare May 12, 2026 16:07
@tombrooks248
tombrooks248 marked this pull request as ready for review May 12, 2026 16:10
Comment thread README.md Outdated
Comment thread README.md Outdated
@tombrooks248
tombrooks248 merged commit 88c410e into main May 13, 2026
1 check passed
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