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

Package and data bumps for April 2024 #193

Closed
wants to merge 4 commits into from

Conversation

devinmatte
Copy link
Member

Motivation

Keeping us up to date on deps, resolving security issues

Changes

  • Bumps npm packages
  • Bumps poetry packages
  • Updates new car stats (to include Type 10 deliveries)

Testing Instructions

@github-actions github-actions bot added the dependencies Changes to dependencies label Apr 19, 2024
@github-actions github-actions bot added the ci/cd Affects or Updates CI/CD label May 17, 2024
static/static_data.json Show resolved Hide resolved
"totalNewDelivered": 24,
"totalNewUndelivered": 0
"totalNewUndelivered": 102
Copy link
Member

Choose a reason for hiding this comment

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

Assume this is now for Type 10s? Originally these numbers were for the Type 9s

Copy link
Member Author

Choose a reason for hiding this comment

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

Yep the roster is now listing Type 10s as undelivered. I know it's a long way off so we can ignore these numbers for now

Copy link
Member

Choose a reason for hiding this comment

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

We should probably call this out? Tough with the Green Line since the 9's were the original new cars but now the 10s are on the horizon. Wdyt?

"totalOldActive": 175,
"totalOldInactive": 22,
"totalOldActive": 178,
"totalOldInactive": 29,
Copy link
Member

Choose a reason for hiding this comment

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

How are this and the number above both higher?

Copy link
Member Author

Choose a reason for hiding this comment

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

Don't know, I can double check the numbers, but that was the counts on the roster when I checked

Copy link
Member

Choose a reason for hiding this comment

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

If that's what the roster says I'm fine to change it, wonder where the discrepancy comes from though

@devinmatte devinmatte closed this Jun 4, 2024
@devinmatte
Copy link
Member Author

superceded by #205

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/cd Affects or Updates CI/CD dependencies Changes to dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants