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.

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

NET-953-tatum-miner-framework #2448

Closed

Conversation

Larskingler
Copy link

This reverts commit 89b21da.

Summary

Please provide a summary of the changes and a motivation if applicable.

Changes

Provide a bullet list of individual changes. Leave this section out if change
set is small and obvious from summary.

Limitations and future improvements

Provide a bullet list or description of known omissions or limitations of the
solution and/or any ideas for future improvement. Leave section ouf if
not applicable.

Checklist before requesting a review

  • Is this a breaking change? If it is, be clear in summary.
  • Read through code myself one more time.
  • Make sure any and all TODO comments left behind are meant to be left in.
  • Has reasonable passing test coverage?
  • Updated changelog if applicable.
  • Updated documentation if applicable.

harbu and others added 15 commits June 9, 2023 12:45
Added maintainTopology service, helper and tests to operatorplugin
Added MaintainOperatorValueService, helper and tests

---------

Co-authored-by: Christoph-Samuel Pitter <christoph-samuel.pitter@streamr.network>
Co-authored-by: Eric Andrews <eric.andrews@streamr.com>
## Summary

Do some refactoring of code.

Changes include:
- Adjust log levels and messages
- Remove unneeded eslint exceptions
- Use `EthereumAddress` type to avoid having to lowercase in many places
- Use `StreamID` type more
- Remove unused code
- Adjust field access modifiers
- Add some missing error handling
@github-actions github-actions bot added ci Related to CI configuration broker Related to Broker Package client Related to Client Package labels Mar 21, 2024
@CLAassistant
Copy link

CLAassistant commented Mar 21, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
3 out of 4 committers have signed the CLA.

✅ samt1803
✅ samigabor
✅ harbu
❌ Larskingler
You have signed the CLA already but the status is still pending? Let us recheck it.

@harbu
Copy link
Contributor

harbu commented Jul 3, 2024

Closing, not sure what this PR was about?

@harbu harbu closed this Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
broker Related to Broker Package ci Related to CI configuration client Related to Client Package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants