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

[ENH] Add COL and HAD to driver dictionaries #609

Merged
merged 3 commits into from
Jul 5, 2024

Conversation

grahamjwhite
Copy link
Contributor

Franco Colapinto (COL) and Isack Hadjar (HAD) participated in FP1 at Silverstone for Williams and Red Bull, respectively. Bearman also participated for Haas, but he already has a colour from previously driving for Ferrari so I left it as is for him.

I spent very little time choosing the colours, feel free to pick something else.

image

@theOehrly
Copy link
Owner

Hadjar is already listed from driving for RB last season (causes the lint error). I'd say we just leave this as is.

Doc build failure is caused externally.

I'll merge and release within the next 2h removing the duplicate Hadjar if that's alright.

@theOehrly theOehrly merged commit 1dc86a0 into theOehrly:master Jul 5, 2024
9 of 10 checks passed
theOehrly added a commit that referenced this pull request Jul 5, 2024
@theOehrly
Copy link
Owner

Thanks! I really need to get the new plotting module implementation done.

(the revert was a misclick)

theOehrly pushed a commit that referenced this pull request Jul 5, 2024
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