FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[Datetime Concept]: Booking Up for Beauty Exercise by colinleach · Pull Request #3567 · exercism/python · GitHub

Repository navigation

[Datetime Concept]: Booking Up for Beauty Exercise - #3567

Open
colinleach wants to merge 1 commit into
exercism:mainfrom
colinleach:booking-up-for-beauty
Open

colinleach wants to merge 1 commit into
exercism:mainfrom
colinleach:booking-up-for-beauty

Conversation

Copy link
Copy Markdown
Contributor

Incomplete draft for discussion. This is a fairly direct port from C#.

Is this suitable for the datetime concept? Usual options:

  • Use with minor tweaks.
  • Use after substantial revision.
  • Scrap it and think of something better.

Some things I'm unhappy about:

  • It's all datetime.datetime, nothing much about time or date
  • Task 2 is locale-specific. It works for me (in the US) and should be OK on the test runner, but will fail for non-English locales. The test file could set the locale to EN-US, but I'm reluctant to mess with users' machine settings unless I'm very confident of restoring it at the end of the test.
  • Task 6 is really lame.

BethanyG changed the title booking-up-for-beauty concept exercise [Datetime Concept]: Booking Up for Beauty Exercise Dec 26, 2023
exercism locked and limited conversation to collaborators Feb 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant


Back | FazBrowse Home | New Git URL