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

Flightcrew taxi ride and score verifier #422

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dmarrazzo
Copy link

Introduced:

  • taxi ride calculation to calculate connection suitability
  • score verifier based on xlsx

@ghost
Copy link

ghost commented Sep 4, 2018

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

3 similar comments
@ghost
Copy link

ghost commented Sep 4, 2018

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

@ghost
Copy link

ghost commented Sep 4, 2018

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

@ghost
Copy link

ghost commented Sep 4, 2018

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

@ghost
Copy link

ghost commented Nov 7, 2018

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

1 similar comment
@ghost
Copy link

ghost commented Nov 7, 2018

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

@triceo
Copy link
Contributor

triceo commented Sep 10, 2019

@ge0ffrey Are we sure this is stale?

@ge0ffrey
Copy link
Contributor

ge0ffrey commented Sep 11, 2019

ok to test

@triceo Ideally, we want to salvage this PR, somehow. Maybe by merging it after a through review - or by rewriting parts (simplifying the design). It's a pity I didn't get around to spending the time on this PR that Donato's work deserves... I did recall I might wanted to try a different design.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants