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

Jump to bottom

Fix #7244 #7226: OPF doesn't take 90 deg turns into account. #7246

Merged
merged 1 commit into from Feb 19, 2019
Merged

Fix #7244 #7226: OPF doesn't take 90 deg turns into account. #7246

merged 1 commit into from Feb 19, 2019

Conversation

J0anJosep
Copy link
Contributor

No description provided.

@michicc
Copy link
Member

michicc commented Feb 18, 2019

You need a comma between the two issue numbers in the commit messages (or maybe even Fix #7224, Fix #7226, but I think there was a change to the CI to get rid of the ugly second Fix).

@michicc
Copy link
Member

michicc commented Feb 18, 2019

Oh, and I think you got your numbers wrong.

@J0anJosep
Copy link
Contributor Author

Yes. I will correct them.

@J0anJosep J0anJosep changed the title Fix #7224 #7226: OPF doesn't take 90 deg turns into account. Fix #7244 #7226: OPF doesn't take 90 deg turns into account. Feb 18, 2019
@LordAro LordAro mentioned this pull request Feb 18, 2019
Merged
@michicc
Copy link
Member

michicc commented Feb 18, 2019

Bad commit checker 😞

Fix #7244, 6ca637b8c1: is supposed to be alright according to the CI log.

@michicc michicc merged commit c4fe46a into OpenTTD:master Feb 19, 2019
@J0anJosep J0anJosep deleted the StupidOPF branch November 22, 2021 21:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@michicc michicc michicc approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@J0anJosep @michicc