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.

Sign up for GitHub

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

Jump to bottom

Add Pace Calculator to SpeedCalculator #15

Closed
4 tasks
Segelzwerg opened this issue Oct 18, 2019 · 2 comments
Closed
4 tasks

Add Pace Calculator to SpeedCalculator #15

Segelzwerg opened this issue Oct 18, 2019 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers Hacktoberfest For everybody how needs some PRs. help wanted Extra attention is needed taken
Milestone
0.2.0

Comments

@Segelzwerg
Copy link
Owner

Add a method that calculates the time need for travelling a given Distance. There are two standard units in the metric system: min/km and min/100m.

  • computePace() returns a Pace Instance
  • Add Pace in IUnits
  • Add inMinutesPerKilometer() to Pace
  • Add inMinPerHundredMeters() to Pace
@Segelzwerg Segelzwerg added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers Hacktoberfest For everybody how needs some PRs. labels Oct 18, 2019
@Segelzwerg Segelzwerg added this to the 0.2.0 milestone Oct 18, 2019
@GledsonAfonso
Copy link
Contributor

Can I try this?

@Segelzwerg
Copy link
Owner Author

yes, and thank you for contributing.

Segelzwerg pushed a commit that referenced this issue Nov 9, 2019
Merge pull request #34 from GledsonAfonso/master
9e77b79 
Adding pace calculation to SpeedCalculator (issue #15)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers Hacktoberfest For everybody how needs some PRs. help wanted Extra attention is needed taken
Projects
None yet
Milestone
0.2.0
Development

No branches or pull requests
2 participants
@GledsonAfonso @Segelzwerg