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

Fixup tests #9

Merged
merged 4 commits into from
Jul 10, 2022
Merged

Fixup tests #9

merged 4 commits into from
Jul 10, 2022

Conversation

tsmock
Copy link
Contributor

@tsmock tsmock commented Jul 6, 2022

This also adds a github workflow, building off of the one I made for the Lanes plugin.

See https://github.com/tsmock/wikipedia/actions/runs/2625888938 for a release run, and https://github.com/tsmock/wikipedia/releases/tag/v1.2.1-test3 for the test release.

Signed-off-by: Taylor Smock <tsmock@fb.com>
@tsmock tsmock force-pushed the fixup-tests branch 9 times, most recently from 5abecfb to 288691f Compare July 6, 2022 22:59
Signed-off-by: Taylor Smock <tsmock@fb.com>
Signed-off-by: Taylor Smock <tsmock@fb.com>
Signed-off-by: Taylor Smock <tsmock@fb.com>
@floscher floscher changed the base branch from master to main July 10, 2022 21:14
@floscher floscher merged commit 03ea278 into JOSM:main Jul 10, 2022
@tsmock
Copy link
Contributor Author

tsmock commented Jul 11, 2022

Werps. I've been fiddling with the workflow a bit. I should have changed this to a draft PR. (see 9e8f607).

@floscher
Copy link
Member

Sorry, but apart from that it looked pretty much ready. And it looks like it works so far, so I guess I'll revert 9e8f607 but leave the rest.

@tsmock
Copy link
Contributor Author

tsmock commented Jul 11, 2022

I had been pushing a known bad version to check and make certain that the test bit of the workflow was working. I'll open a separate PR with the test fixes (they were in this pr originally, but while I was testing the workflow, they were temporarily removed).

EDIT: NVM, it looks like you fixed the failing tests already.

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