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

docs: Remove obsolete LoggingNetwork documentation from README #785

Merged
merged 2 commits into from
Dec 30, 2022

Conversation

allanparsons
Copy link
Contributor

@allanparsons allanparsons commented Dec 30, 2022

According to Change Log documentation, logging_network support was removed in version 2.0.0. README documentation was never updated to reflect this removal. This PR resolves that discrepancy between docs and changelog.

This PR replaces PR #783

@allanparsons
Copy link
Contributor Author

This PR replaces PR #783 , which was closed due to CLA signing issues related to github username discepencies

@coveralls
Copy link

Pull Request Test Coverage Report for Build 3807491817

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 93.485%

Totals Coverage Status
Change from base Build 3801423655: 0.0%
Covered Lines: 3415
Relevant Lines: 3653

💛 - Coveralls

@lukaszsocha2 lukaszsocha2 changed the title docs: Remove obsoleteLoggingNetwork docs from README docs: Remove obsolete LoggingNetwork docs from README Dec 30, 2022
@lukaszsocha2 lukaszsocha2 changed the title docs: Remove obsolete LoggingNetwork docs from README docs: Remove obsolete LoggingNetwork docs from README Dec 30, 2022
@lukaszsocha2 lukaszsocha2 changed the title docs: Remove obsolete LoggingNetwork docs from README docs: Remove obsolete LoggingNetwork documentation from README Dec 30, 2022
@lukaszsocha2 lukaszsocha2 merged commit ff6fa79 into box:main Dec 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants