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

Fix markdownlint in site #1499

Merged
merged 2 commits into from
Jan 24, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,9 +43,9 @@ repos:
# Rules are in .markdownlint-cli2.yaml file
# See https://github.com/DavidAnson/markdownlint/blob/main/doc/Rules.md for rule descriptions
- repo: https://github.com/DavidAnson/markdownlint-cli2
rev: v0.11.0
rev: v0.12.1
hooks:
- id: markdownlint-cli2-fix
lucacome marked this conversation as resolved.
Show resolved Hide resolved
- id: markdownlint-cli2

# Rules are in .yamllint.yaml file
# See https://yamllint.readthedocs.io/en/stable/rules.html# for rule descriptions
Expand All @@ -55,5 +55,5 @@ repos:
- id: yamllint

ci:
skip: [golangci-lint-full, prettier, markdownlint-cli2-fix, yamllint]
skip: [golangci-lint-full, prettier, markdownlint-cli2, yamllint]
autofix_prs: false
4 changes: 3 additions & 1 deletion site/mdlint_conf.json → site/.markdownlint.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,5 +16,7 @@
},
"MD046": false,
"MD001": false,
"MD049": false
"MD049": false,
"MD055": false,
"MD056": false
sjberman marked this conversation as resolved.
Show resolved Hide resolved
}
2 changes: 1 addition & 1 deletion site/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ link-check:
${MARKDOWNLINKCHECK} $(shell find content -name '*.md')

lint-markdown:
${MARKDOWNLINT} -c .markdownlint.yaml -- content
${MARKDOWNLINT} -c .markdownlint.json -- content

# Commands used by Netlify CI
hugo-mod:
Expand Down
2 changes: 1 addition & 1 deletion site/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ If you have [Docker](https://www.docker.com/get-started/) installed, there are f
The configuration files are as follows:

- *Hugo*: `config/default/config.toml`
- *markdownlint-cli*: `mdlint_conf.json`
- *markdownlint-cli*: `.markdownlint.json`
- *markdown-link-check* `md-linkcheck-config.json`

## Repository guidelines
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@ At a high level, the process looks like this:
1. We create a gateway resource for our domain (cafe.example.com) and configure cert-manager integration using an annotation.
1. This starts the certificate issuance process – cert-manager contacts Let’s Encrypt to obtain a certificate, and Let’s Encrypt starts the ACME challenge. As part of this challenge, cert-manager creates a temporary HTTPRoute resource which directs the traffic through NGINX Gateway Fabric to verify we control the domain name in the certificate request.
1. Once the domain has been verified, the certificate is issued. Cert-manager stores the keypair in a Kubernetes secret that is referenced by the gateway resource. As a result, NGINX is configured to terminate HTTPS traffic from clients using this signed keypair.
1. We deploy our application and our HTTPRoute which defines our routing rules. The routing rules defined configure NGINX to direct requests to https://cafe.example.com/coffee to our coffee-app application, and to use the HTTPS listener defined in our gateway resource.
1. When the client connects to https://cafe.example.com/coffee, the request is routed to the coffee-app application and the communication is secured using the signed keypair contained in the cafe-secret secret.
1. We deploy our application and our HTTPRoute which defines our routing rules. The routing rules defined configure NGINX to direct requests to `https://cafe.example.com/coffee` to our coffee-app application, and to use the HTTPS listener defined in our gateway resource.
1. When the client connects to `https://cafe.example.com/coffee`, the request is routed to the coffee-app application and the communication is secured using the signed keypair contained in the cafe-secret secret.
1. The certificate will be automatically renewed when it is close to expiry, the secret will be updated using the new certificate, and NGINX Gateway Fabric will dynamically update the keypair on the filesystem used by NGINX for HTTPS termination once the secret is updated.

## Securing Traffic
Expand Down Expand Up @@ -205,7 +205,7 @@ spec:

## Testing

To test everything has worked correctly, we can use curl to the navigate to our endpoint, for example, https://cafe.example.com/coffee. To verify using curl, we can use the `-v` option to increase verbosity and inspect the presented certificate.
To test everything has worked correctly, we can use curl to the navigate to our endpoint, for example, `https://cafe.example.com/coffee`. To verify using curl, we can use the `-v` option to increase verbosity and inspect the presented certificate.

```shell
curl https://cafe.example.com/coffee -v
Expand Down
2 changes: 1 addition & 1 deletion site/content/reference/technical-specifications.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,4 @@ docs: "DOCS-000"

See the NGINX Gateway Fabric technical specifications page:

https://github.com/nginxinc/nginx-gateway-fabric#technical-specifications
<https://github.com/nginxinc/nginx-gateway-fabric#technical-specifications>
2 changes: 1 addition & 1 deletion site/content/releases.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,4 @@ docs: "DOCS-1359"

See the NGINX Gateway Fabric changelog page:

https://github.com/nginxinc/nginx-gateway-fabric/blob/main/CHANGELOG.md
<https://github.com/nginxinc/nginx-gateway-fabric/blob/main/CHANGELOG.md>
Loading