fix req and dev readme with protobuf step

[lalmei]

Still need your input on where to modify and change.

[coveralls]

Pull Request Test Coverage Report for Build 753121931

• 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 82.966%

---

Totals
Change from base Build 737102552:	0.0%
Covered Lines:	3190
Relevant Lines:	3704

---

💛 - Coveralls

[sourcery-ai]

Sourcery Code Quality Report

Merging this PR leaves code quality unchanged.

Quality metrics	Before	After	Change
Complexity	5.86 ⭐	5.86 ⭐	0.00
Method Length	138.00 😞	138.00 😞	0.00
Working memory	14.70 😞	14.70 😞	0.00
Quality	41.84% 😞	41.84% 😞	0.00%

Other metrics	Before	After	Change
Lines	277	277	0

Changed files	Quality Before	Quality After	Quality Change
docs/conf.py	40.49% 😞	40.49% 😞	0.00%
src/whylogs/_version.py	100.00% ⭐	100.00% ⭐	0.00%

Here are some functions in these files that still need a tune-up:

File	Function	Complexity	Length	Working Memory	Quality	Recommendation

Legend and Explanation

The emojis denote the absolute quality of the code:

• ⭐ excellent
• 🙂 good
• 😞 poor
• ⛔ very poor

The 👍 and 👎 indicate whether the quality has improved or gotten worse with this pull request.

---

Please see our documentation here for details on how these metrics are calculated.

We are actively working on this report - lots more documentation and extra metrics to come!

Let us know what you think of it by mentioning @sourcery-ai in a comment.

[andyndang]

we shouldn't have sphinx devs in our dev dependencies. This is for documentation dependencies and they are separate

[lalmei]

i disagree, my understanding is that this was included as a full dependency for a developer working on this project. which what the team has been asking for. otherwise lets just remove this file.

[lalmei]

but agree it shouldn't be part of the main requirements

[cswarth]

LGTM