test bot (#1097)

* update action

* typo here

* redirect here

* update

* Fix typos

* Revert "Fix typos"

This reverts commit 9e775f57c21e85cc87f37ca51bc706fbe607f4da.

* merge all into one

* Add fixes

* check branch not commit

* update script

* Add fixes

* final rearrangement?

* Add safe redirects

* Fix typos

* remove useless changes

Co-authored-by: PostHog <hey@posthog.com>

.github/workflows/checks.yml

@@ -24,18 +24,24 @@ jobs:
                   python -m pip install -U pip
                   python -m pip install git+https://github.com/yakkomajuri/codespell.git
 
-            - name: Correct spelling
+            - name: Configure Git
               run: |
-                  codespell ./contents -w -S "./contents/images/*,./contents/docs/_media/*" -q 8 -I ./.codespellignore
+                  git fetch origin master:master
                   git config user.name PostHog Bot
                   git config user.email hey@posthog.com
-                  git add .
-                  [ "$(git status -s)" = "" ] && exit 0 || git commit -m "Fix typos" && git push
 
             - name: Add safe redirects
               run: |
-                  git fetch origin master:master
-                  export PR_DIFF=$(git diff master)
-                  python3 ./scripts/safe_rename.py
+                  PR_DIFF=$(git diff ${{ github.head_ref }} ${{ github.base_ref }}) python3 ./scripts/safe_rename.py
                   git add .
-                  [ "$(git status -s)" = "" ] && exit 0 || git commit -m "Add redirects" && git push
+                  [ "$(git status -s)" = "" ] && exit 0 || git commit -m "Add safe redirects"
+
+            - name: Fix typos
+              run: |
+                  codespell ./contents -w -S "./contents/images/*,./contents/docs/_media/*" -q 8 -I ./.codespellignore
+                  git add .
+                  [ "$(git status -s)" = "" ] && exit 0 || git commit -m "Fix typos"
+
+            - name: Push changes
+              run: |
+                  git push

contents/docs/features/actions.md

@@ -109,5 +109,4 @@ If you have buttons across your site, all saying ‘Sign Up’, you can track AL
 
 If you choose to match the URL as well, it will track any button on that page if it contains the ‘Sign Up’ text (which could be more than one button).
 
-If you have multiple sign up buttons on the same page, all with the same text and you want to track events relating to just one of them, you can use ‘Selector’. If they are technically the same element type **and** have the same classes, then you need to add a new class, `id` or `data-attr` to the button you wish to track separately, and use this to identify it.
-
+If you have multiple sign up buttons on the same page, all with the same text and you want to track events relating to just one of them, you can use ‘Selector’. If they are technically the same element type **and** have the same classes, then you need to add a new class, `id` or `data-attr` to the button you wish to track separately, and use this to identify it.

netlify.toml

@@ -1,4 +1,4 @@
 [[redirects]]
   from = "/plugins/*"
   to = "/plugins"
-  status = 200
+  status = 200

scripts/safe_rename.py

@@ -23,7 +23,7 @@ if len(from_paths) > 0 and len(from_paths) == len(to_paths):
     
     new_redirects = ''
 
-    for i in range(len(from_paths)-1):
+    for i in range(len(from_paths)):
         if from_paths[i] not in netlify_config_text:
             new_redirects += redirect_text.format(from_paths[i], to_paths[i])