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

Use correct function name #1068

Merged
merged 1 commit into from
Nov 10, 2023
Merged

Use correct function name #1068

merged 1 commit into from
Nov 10, 2023

Conversation

fiveop
Copy link
Contributor

@fiveop fiveop commented Oct 13, 2023

The function is first called get_total_cars, which fits to the problem description, then get_total. This pull request fixes that.

@github-actions
Copy link

🆗 Pre-flight checks passed 😃

This pull request has been checked and contains no modified workflow files, spoofing, or invalid commits.

It should be safe to Approve and Run the workflows that need maintainer approval.

Copy link
Member

@ineelhere ineelhere left a comment

Choose a reason for hiding this comment

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

Thanks for the fix @fiveop 🚀
Best wishes! 🙂

@ineelhere ineelhere assigned ineelhere and unassigned ineelhere Nov 10, 2023
@ineelhere ineelhere merged commit cf9bc3f into swcarpentry:main Nov 10, 2023
2 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 10, 2023
Auto-generated via {sandpaper}
Source  : cf9bc3f
Branch  : main
Author  : Indraneel Chakraborty <[email protected]>
Time    : 2023-11-10 14:30:42 +0000
Message : Merge pull request #1068 from fiveop/total_cars

Use correct function name
github-actions bot pushed a commit that referenced this pull request Nov 10, 2023
Auto-generated via {sandpaper}
Source  : dd80e0c
Branch  : md-outputs
Author  : GitHub Actions <[email protected]>
Time    : 2023-11-10 14:33:13 +0000
Message : markdown source builds

Auto-generated via {sandpaper}
Source  : cf9bc3f
Branch  : main
Author  : Indraneel Chakraborty <[email protected]>
Time    : 2023-11-10 14:30:42 +0000
Message : Merge pull request #1068 from fiveop/total_cars

Use correct function name
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