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

fix(deps): remove alpha dependencies #688

Merged

Conversation

daniel-heppner-ibigroup
Copy link
Contributor

@daniel-heppner-ibigroup daniel-heppner-ibigroup commented Nov 30, 2023

Remove dependencies on the alpha branch.

Copy link
Collaborator

@binh-dam-ibigroup binh-dam-ibigroup left a comment

Choose a reason for hiding this comment

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

Could you fix the TypeScript errors, please @daniel-heppner-ibigroup?

@daniel-heppner-ibigroup
Copy link
Contributor Author

What the heck, where did these completely unrelated typescript errors come from? 😠 I'll take a look

@daniel-heppner-ibigroup
Copy link
Contributor Author

@binh-dam-ibigroup Looks like this was a bug from another PR to types which was only caught here by upgrading the package version in core utils.

Copy link
Collaborator

@miles-grant-ibigroup miles-grant-ibigroup left a comment

Choose a reason for hiding this comment

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

Great fixes thanks!

@miles-grant-ibigroup miles-grant-ibigroup merged commit 8d1d251 into opentripplanner:master Nov 30, 2023
2 checks passed
@miles-grant-ibigroup miles-grant-ibigroup deleted the nearby-view-no-alpha branch November 30, 2023 18:51
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.

3 participants