Some cleanup

Author: andreiborza

docs/development.md

@@ -40,14 +40,13 @@ Telemetry for internal development is collected using `@sentry/node`, see `src/t
 Members of this repo will not have to set anything up since [the integration](https://sentry-ecosystem.sentry.io/settings/developer-settings/end-to-end-action-release-integration-416eb2/) is already set-up. Just open the PR and you will see [a release created](https://sentry-ecosystem.sentry.io/releases/?project=4505075304693760) for your PR.
 
 > [!WARNING]  
-> After you create a branch **ALWAYS** run `yarn bump-docker-tag-from-branch`.  
-> This is **very important**. You should avoid publishing changes to an already existing docker image,
-> especially to docker images tagged with release versions, e.g. `1.11.0`.
->
-> **Never** write a version tag into `action.yml` yourself, let the release process handle this.
+> After you create a branch **ALWAYS** run `yarn set-docker-tag-from-branch`.  
+> This is **very important**. You should avoid publishing changes to an already existing docker image.
+> 
+> Docker tags in action.yml that match `MAJOR.MINOR.PATCH` will be rejected by the CI on pull requests.
 
 1. Create a branch
-2. Run `yarn bump-docker-tag-from-branch` to set a docker tag based on your github branch name. This is important so that the action gets its own Docker image, allowing you to test the action in a different repo.
+2. Run `yarn set-docker-tag-from-branch` to set a docker tag based on your github branch name. This is important so that the action gets its own Docker image, allowing you to test the action in a different repo.
 3. Make changes
 4. If possible, add unit and E2E tests (inside `.github/workflows/build.yml`)
 5. Run `yarn install` to install deps

package.json

@@ -5,15 +5,15 @@
   "description": "GitHub Action for creating a release on Sentry",
   "main": "dist/index.js",
   "scripts": {
-    "start": "node dist/index.js",
+    "all": "yarn run format && yarn run lint && yarn run build && yarn test",
     "build": "ncc build src/main.ts -e @sentry/cli",
     "format": "prettier --write **/*.ts **/*.md",
     "format-check": "prettier --check **/*.ts **/*.md",
     "lint": "eslint src/**/*.ts",
-    "test": "jest",
-    "all": "yarn run format && yarn run lint && yarn run build && yarn test",
-    "bump-docker-tag": "./scripts/set-docker-tag.sh",
-    "bump-docker-tag-from-branch": "./scripts/set-branch-docker-tag.sh"
+    "set-docker-tag": "./scripts/set-docker-tag.sh",
+    "set-docker-tag-from-branch": "./scripts/set-docker-tag-from-branch.sh",
+    "start": "node dist/index.js",
+    "test": "jest"
   },
   "repository": {
     "type": "git",

scripts/craft-pre-release.sh

@@ -18,4 +18,4 @@ npm version "${NEW_VERSION}"
 yarn install && yarn build
 
 # Update the docker tag in action.yml
-yarn bump-docker-tag "${NEW_VERSION}"
+yarn set-docker-tag "${NEW_VERSION}"