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: improve markdown quality #498

Merged

Conversation

aepfli
Copy link
Member

@aepfli aepfli commented Mar 14, 2023

This PR

Adding markdownlint for local execution and GitHub action
via makefile. Additionally, we provide an auto-fix target to fix the documentation quickly.

  • adds this new feature

Related Issues

First iteration for #497

Notes

Besides the checked int markdownlint configuration, I also applied the rule max-one-sentence-per-line

aepfli added 2 commits March 14, 2023 08:42
Adding markdownlint for local execution and GitHub action
via makefile. Additionally we provide a autofix target to
easily fix documentation.

Signed-off-by: Simon Schrottner <simon.schrottner@dynatrace.com>
Signed-off-by: Simon Schrottner <simon.schrottner@dynatrace.com>
@aepfli
Copy link
Member Author

aepfli commented Mar 14, 2023

image

as a sneak peak - why i need some of those changes - eg. closing details tags missing etc.

@codecov
Copy link

codecov bot commented Mar 14, 2023

Codecov Report

Merging #498 (0a35921) into main (0b5ccbd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #498   +/-   ##
=======================================
  Coverage   63.39%   63.39%           
=======================================
  Files          15       15           
  Lines        1923     1923           
=======================================
  Hits         1219     1219           
  Misses        639      639           
  Partials       65       65           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@beeme1mr beeme1mr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great start, thank you!

Signed-off-by: Simon Schrottner <simon.schrottner@dynatrace.com>
@beeme1mr beeme1mr merged commit c77fa37 into open-feature:main Mar 15, 2023
@github-actions github-actions bot mentioned this pull request Mar 15, 2023
toddbaert pushed a commit that referenced this pull request Mar 20, 2023
🤖 I have created a release *beep* *boop*
---


<details><summary>flagd: 0.4.5</summary>

##
[0.4.5](flagd/v0.4.4...flagd/v0.4.5)
(2023-03-20)


### 📚 Documentation

* improve markdown quality
([#498](#498))
([c77fa37](c77fa37))


### ✨ New Features

* grpc connection options to flagd configuration options
([#532](#532))
([aa74951](aa74951))
* Introduce flagd kube proxy
([#495](#495))
([440864c](440864c))
</details>

<details><summary>kube-flagd-proxy: 0.1.2</summary>

##
[0.1.2](kube-flagd-proxy/v0.1.1...kube-flagd-proxy/v0.1.2)
(2023-03-20)


### ✨ New Features

* Introduce flagd kube proxy
([#495](#495))
([440864c](440864c))
</details>

<details><summary>core: 0.4.5</summary>

##
[0.4.5](core/v0.4.4...core/v0.4.5)
(2023-03-20)


### 🐛 Bug Fixes

* **deps:** update kubernetes packages to v0.26.3
([#533](#533))
([6ddd5b2](6ddd5b2))
* **deps:** update module github.com/open-feature/open-feature-operator
to v0.2.31 ([#527](#527))
([7928130](7928130))
* **deps:** update module google.golang.org/protobuf to v1.29.1
[security] ([#504](#504))
([59db0bb](59db0bb))
* **deps:** update module google.golang.org/protobuf to v1.30.0
([#499](#499))
([f650338](f650338))


### ✨ New Features

* grpc connection options to flagd configuration options
([#532](#532))
([aa74951](aa74951))
* Introduce flagd kube proxy
([#495](#495))
([440864c](440864c))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Michael Beemer <beeme1mr@users.noreply.github.com>
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.

2 participants