Skip to content

Docs: Add detailed information on integration testing #1593

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

Merged
merged 5 commits into from
Jun 24, 2025

Conversation

harahu
Copy link
Contributor

@harahu harahu commented Jun 11, 2025

Description, Motivation & Context

I had some trouble figuring out how to test schema changes locally on my machine. @tordans helpfully figured out how to do so and provided me with a solution that works.

We felt it would be appropriate to document this workflow, and this is my attempt at doing so.

Copy link

🍱 Your pull request preview is ready

Please use this preview to check your changes. Ideally use the test documentation template and document your test results by commenting on the PR. This will speed up the review process for everyone.

FYI, once this PR is merged, you can use the iD Editor Preview to test your changes in interaction with all other changes.

harahu and others added 3 commits June 18, 2025 15:20
Copy link
Collaborator

@tordans tordans left a comment

Choose a reason for hiding this comment

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

I move the section up a bit and added a bit more structure to the two options.
The wording for the PR preview is the same as in https://github.com/openstreetmap/id-tagging-schema/pull/1606/files which will make it easier to find both to keep them updated.

I would merge this in a few days, if nothing comes up…

@tordans tordans added documentation Improvements or additions to documentation waiting-ready-to-merge Ready to merge, but let's wait a few days for possible feedback. labels Jun 19, 2025
@tordans tordans changed the title Add more detailed information on integration testing Docs: Add detailed information on integration testing Jun 19, 2025
@tordans tordans merged commit 9fa499b into openstreetmap:main Jun 24, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation waiting-ready-to-merge Ready to merge, but let's wait a few days for possible feedback.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants