mimir icon indicating copy to clipboard operation
mimir copied to clipboard

Issue 1330 documentation changes

Open konstantinos-null opened this issue 3 years ago • 1 comments

What this PR does

Opened this PR to tackle issue 1330 and fix up the related documentation points.

Which issue(s) this PR fixes or relates to

Fixes #1330

Checklist

  • [ ] Tests updated
  • [ ] Documentation added
  • [ ] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

konstantinos-null avatar Oct 17 '22 18:10 konstantinos-null

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Oct 17 '22 18:10 CLAassistant

Sorry, clicked the wrong button trying to run the workflow

56quarters avatar Oct 18 '22 18:10 56quarters

Sorry, clicked the wrong button trying to run the workflow

Ok! Anything i should be doing on my part?

konstantinos-null avatar Oct 18 '22 18:10 konstantinos-null

Sorry, clicked the wrong button trying to run the workflow

Ok! Anything i should be doing on my part?

Nope! Feel free to work on this as you see fit and mark it ready when you'd like review

56quarters avatar Oct 18 '22 19:10 56quarters

Hey, I think that most parts that could be altered are done so, the ones remaining were in sentences where if I would change them it would seem too weird I think. Still if there are any suggestions I am open!

konstantinos-null avatar Oct 19 '22 18:10 konstantinos-null

@pracucci, can you add some rationale as to why there are so many recommendations? I think a little rationale can go a long way.

osg-grafana avatar Nov 24 '22 11:11 osg-grafana

Hey @osg-grafana, first of all thank you for all the valuable feedback, it's nice getting a bit more insight on the project. I'll go over the changes in the next couple of days and push some change for either approval or more feedback!

konstantinos-null avatar Nov 26 '22 19:11 konstantinos-null

The CHANGELOG has just been cut to prepare for the next Mimir release. Please rebase main and eventually move the CHANGELOG entry added / updated in this PR to the top of the CHANGELOG document. Thanks!

replay avatar Nov 29 '22 14:11 replay

@konstantinos-null, @pracucci and I will incorporate the style changes in subsequent PRs as those files are touched. Thank you for providing feedback that we can refer back to.

osg-grafana avatar Jan 24 '23 09:01 osg-grafana