Skip to content

Commit 63fde8d

Browse files
authored
Merge pull request #3705 from manics/pre-commit-fixes
Pre commit fixes: switch to supported mirrors-prettier, workaround beautysh bug
2 parents e920203 + 6677898 commit 63fde8d

File tree

12 files changed

+8
-29
lines changed

12 files changed

+8
-29
lines changed

.pre-commit-config.yaml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,16 @@ repos:
5252
rev: v6.2.1
5353
hooks:
5454
- id: beautysh
55+
additional_dependencies:
56+
# On recent Python versions beautysh fails with
57+
# ModuleNotFoundError: No module named 'pkg_resources'
58+
# This can be removed if https://github.com/lovesegfault/beautysh/pull/251
59+
# is released
60+
- setuptools
5561

5662
# Autoformat: markdown, yaml (but not helm templates)
57-
- repo: https://github.com/pre-commit/mirrors-prettier
58-
rev: v4.0.0-alpha.8
63+
- repo: https://github.com/rbubley/mirrors-prettier
64+
rev: v3.6.2
5965
hooks:
6066
- id: prettier
6167

RELEASE.md

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@ Also the images we build are based on some image specified in the `FROM` stateme
4949
## Pre-release iteration
5050

5151
- Update `docs/source/changelog.md`
52-
5352
- [ ] Generate a list of PRs using [executablebooks/github-activity](https://github.com/executablebooks/github-activity)
5453
```bash
5554
github-activity --output github-activity-output.md --since <last tag> jupyterhub/zero-to-jupyterhub-k8s
@@ -60,7 +59,6 @@ Also the images we build are based on some image specified in the `FROM` stateme
6059
- [ ] Summarize the release changes
6160

6261
- Tag a x.y.z-beta.1 release
63-
6462
- [ ] Create and push a git tag
6563
```bash
6664
git checkout main
@@ -75,15 +73,13 @@ Also the images we build are based on some image specified in the `FROM` stateme
7573
## Final release
7674

7775
- Update `docs/source/changelog.md`
78-
7976
- [ ] Generate a list of merged PRs and a list of contributors and update the changelog.
8077
```bash
8178
github-activity --output github-activity-output.md --since <last tag> jupyterhub/zero-to-jupyterhub-k8s
8279
```
8380
- [ ] Link out to the downstream projects within the JupyterHub org to celebrate work done there as well.
8481

8582
- Release
86-
8783
- [ ] Create and push a git tag.
8884

8985
```bash

docs/source/administrator/optimization.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -270,7 +270,6 @@ following:
270270

271271
If you need help on how to do this, please refer to your cloud providers
272272
documentation. A node pool may be called a node group.
273-
274273
- The label: `hub.jupyter.org/node-purpose=user`
275274

276275
**NOTE**: Cloud providers often have their own labels, separate from

docs/source/changelog.md

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1081,7 +1081,6 @@ and small bugfixes will increment the three version numbers.
10811081
you if you don't specify them.
10821082

10831083
- **Smoother helm upgrades**
1084-
10851084
- `prePuller.hook.pullOnlyOnChanges` is now available and enabled by default,
10861085
which only intercepts a `helm upgrade` by pulling images if they have
10871086
changed since the last upgrade.
@@ -1131,7 +1130,6 @@ followed these instructions between `0.7.0-beta.1` and `0.11.1`, please see the
11311130
cluster.
11321131

11331132
The most common validation errors are:
1134-
11351133
- _Unrecognized config values_
11361134

11371135
For example if you have misspelled something.
@@ -4148,7 +4146,6 @@ Deployer UX fixes.
41484146
- No need to restart hub manually after some changes - it is
41494147
automatically restarted now. You can disable an automatic
41504148
restart of hub after an upgrade with the following:
4151-
41524149
1. Finding out the current helm release's revision
41534150
2. Adding '--set revisionOverride=<current-revision>' to your
41544151
upgrade command.

docs/source/jupyterhub/installation.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,6 @@ can try with `nano config.yaml`.
7575
```
7676

7777
where:
78-
7978
- `<helm-release-name>` refers to a [Helm release name](https://helm.sh/docs/glossary/#release), an identifier used to
8079
differentiate chart installations. You need it when you are changing or
8180
deleting the configuration of this chart installation. If your Kubernetes

docs/source/jupyterhub/uninstall.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,6 @@ before doing the cloud provider specific setup.
6666
has been deleted.
6767

6868
At a minimum, check the following under the Hamburger (left top corner) menu:
69-
7069
1. Compute -> Compute Engine -> Disks
7170
2. Compute -> Kubernetes Engine -> Clusters
7271
3. Tools -> Container Registry -> Images

docs/source/kubernetes/amazon/step-zero-aws-eks.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ This guide uses AWS to set up a cluster. This mirrors the steps found at [Gettin
1212
Instructions for creating a role can be found
1313
[here](https://docs.aws.amazon.com/IAM/latest/UserGuide/id_roles_create.html).
1414
It should have the following policies:
15-
1615
- AmazonEKSClusterPolicy
1716
- AmazonEKSServicePolicy
1817
- AmazonEC2ContainerRegistryReadOnly

docs/source/kubernetes/amazon/step-zero-aws.md

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ template you will use to setup and shape your cluster.
1717
resources on AWS. Instructions for creating a role can be found
1818
[here](https://docs.aws.amazon.com/IAM/latest/UserGuide/id_roles_create.html).
1919
The following policies are required:
20-
2120
- AmazonEC2FullAccess
2221
- IAMFullAccess
2322
- AmazonS3FullAccess
@@ -41,7 +40,6 @@ template you will use to setup and shape your cluster.
4140
[here](https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/connect-linux-inst-ssh.html).
4241

4342
4. Install kops and kubectl on your CI host
44-
4543
- Follow the instructions here: <https://github.com/kubernetes/kops/blob/HEAD/docs/install.md>
4644

4745
5. Choose a cluster name:
@@ -172,7 +170,6 @@ template you will use to setup and shape your cluster.
172170
You should see a list of two nodes, each beginning with `ip`.
173171
174172
If you want to use kubectl and helm locally:
175-
176173
- run the following on CI host: `kops export kubecfg`
177174
- copy the contents of `~/.kube/config` to the same place on your local system
178175
@@ -189,7 +186,6 @@ template you will use to setup and shape your cluster.
189186
Instead we need to follow this guide: <https://github.com/kubernetes/kops/blob/HEAD/docs/examples/kops-tests-private-net-bastion-host.md#adding-a-bastion-host-to-our-cluster>
190187
191188
At this point there are a few public endpoints left open which need to be addressed
192-
193189
- Bastion ELB security group defaults to access from 0.0.0.0
194190
- API ELB security group defaults to access from 0.0.0.0
195191

docs/source/kubernetes/digital-ocean/step-zero-digital-ocean.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ This page describes the commands required to setup a Kubernetes cluster using th
88
If you prefer to use the Digital Ocean portal see the [Digital Ocean Get Started](https://www.digitalocean.com/products/kubernetes).
99

1010
1. Prepare your Digital Ocean shell environment.
11-
1211
1. **Install command-line tools locally**. You'll need at least v1.13.0.
1312

1413
You can either follow the [installation instructions](https://github.com/digitalocean/doctl/blob/main/README.md) or use the commands below:

docs/source/kubernetes/google/step-zero-gcp.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ your google cloud account.
2424
command line interfaces on your own computer's terminal. We recommend
2525
starting out by using the web based terminal. Choose one set of instructions
2626
below.
27-
2827
1. **Use a web based terminal:**
2928

3029
Start _Google Cloud Shell_ from [console.cloud.google.com](https://console.cloud.google.com) by clicking the button shown below.
@@ -38,7 +37,6 @@ your google cloud account.
3837
```
3938
4039
2. **Use your own computer's terminal:**
41-
4240
1. Download and install the `gcloud` command line tool at its [install
4341
page](https://cloud.google.com/sdk/docs/install). It will help you
4442
create and communicate with a Kubernetes cluster.

0 commit comments

Comments
 (0)