Added Study Room Functions to to Library Module

[ghosteau]

• Added a function to check reserved Hillman library study room times, tied to which room is specifically reserved
• Added another function to check the total amount of reserved rooms in the Hillman library

[ghosteau]

Some very slight spacing changes may have been made to lab module because I wanted to revert it back on my fork. Just ignore all changes outside of the library module

[tianyizheng02]

Please run black . in your local git repo to fix the formatting issues

[ghosteau]

I ran black through cmd, let me know if there is anything else that needs done.

[ghosteau]

I think I dealt with the merger conflicts -- I elected to choose the segments that were formatted with black to hopefully make readability for the API better for some portions. Do let me know if there is anything else, or anything I missed.

[tianyizheng02]

I think all the biggest issues (syntax errors, etc) have been fixed, so we can now let GitHub run our repo workflows on this PR before I comment on anything else in the code

[tianyizheng02]

So there were still more errors with black cuz we run black with extra command-line args and that was messing with things... I ran black again locally and pushed a commit to get the workflows to pass, sorry about all the fuss

[tianyizheng02]

The GitHub workflow is failing because of formatting issues. For future contributions, I highly, highly recommend that you set up pre-commit and other tooling that we use in this repo (I have a write-up on how to set up everything in #170, but it hasn't been merged yet). Pre-commit automatically checks for certain issues and runs tools like black before you make a git commit, so it streamlines everything and you don't need to worry about formatting and stuff yourself.

[ghosteau]

I wrote the unit tests and I believe I cleaned up all of the slight errors with the hints and spacing. Let me know if there is anything else that needs done!

For the unit tests I did make a new class for the new extension of the library module. If you want, I can add it to the old class, but I figured it would be easier to separate them since they do use different links and different mock JSONs.

I also had to update the path to be SAMPLE_PATH = Path(__file__).parent / "samples" instead, because for some reason the tests were encountering issues using the previous method.

[tianyizheng02]

I think it looks good, let's run the test workflow and see if there are any issues

[tianyizheng02]

black is throwing an error again because of some minor formatting stuff (even though personally I think your code is fine)... see my comments below

[tianyizheng02]

Everything looks good! The only thing left is to update all the type hints to use the built-in types rather than the typing library types (I cleaned up most of the types throughout the repo in a different PR, so it's best that we keep things consistent now). I'll handle this edit myself because GitHub's suggestion system limits where I can leave suggestions.

[ghosteau]

Sounds good! I appreciate all of the help with the formatting especially!

[tianyizheng02]

All workflows/tests are passing, and I think everything looks good! Thanks for your work and for your patience with the tooling. As mentioned earlier, I highly recommend learning how to install and use pre-commit and other tools (see CONTRIBUTING.md), because that'll make the PR process much easier and more straightforward for everyone.