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

Update testing #8

Merged
merged 4 commits into from
Jul 19, 2024
Merged

Update testing #8

merged 4 commits into from
Jul 19, 2024

Conversation

jacobbieker
Copy link
Member

@jacobbieker jacobbieker commented Jul 19, 2024 •
edited
Loading

Fix #6

Pull Request

Description

Updates tests to be more useful for the codec.

Fixes #

How Has This Been Tested?

Unit test

  • 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 Jul 19, 2024
@jacobbieker jacobbieker marked this pull request as ready for review July 19, 2024 07:46
@jacobbieker jacobbieker merged commit e521a5a into main Jul 19, 2024
3 checks passed
@jacobbieker jacobbieker deleted the jacob/fix-tests branch July 19, 2024 07:49
@peterdudfield
Copy link
Contributor

@all-contributors please add @jacobbieker for code

@allcontributors allcontributors bot mentioned this pull request Jul 19, 2024
Merged
@allcontributors AllContributors
Copy link
Contributor

@peterdudfield

I've put up a pull request to add @jacobbieker! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@peterdudfield peterdudfield Awaiting requested review from peterdudfield

Assignees

@jacobbieker jacobbieker

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Get tests working
2 participants
@jacobbieker @peterdudfield