Skip to content
This repository was archived by the owner on Jun 2, 2025. It is now read-only.

Add more tests/linting updates #40

Merged
merged 21 commits into from
Sep 1, 2022
Merged

Add more tests/linting updates #40

merged 21 commits into from
Sep 1, 2022

Conversation

jacobbieker
Copy link
Member

Pull Request

Description

Adds more unit tests and adds docstrings/fixes some linting errors in parts of the code

Fixes #

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.
Please also list any relevant details for your test configuration

  • Yes

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

  • Yes

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked my code and corrected any misspellings

@jacobbieker jacobbieker self-assigned this Sep 1, 2022
@jacobbieker jacobbieker marked this pull request as ready for review September 1, 2022 10:28
@jacobbieker jacobbieker merged commit 8055507 into main Sep 1, 2022
@jacobbieker jacobbieker deleted the jacob/more-tests branch September 1, 2022 10:31
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