-
Notifications
You must be signed in to change notification settings - Fork 25
Fix the Issue of Code sanitisation #159 and Updated the contributting.md #238
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
Changes from all commits
b5d9e43
852e350
f9a72d0
9143a2c
560ceed
88dedfe
3921e57
2fa3a60
48edb0e
730e00a
79db946
01e95a6
cfe0068
0b20c6a
84bb3ed
825febb
b4a31da
d2f4139
9b3ccac
38e4da1
8138218
2ab0f0c
be943a6
2452ebe
33f4865
cf56585
bd4aced
2b82d79
34f1ad3
d62aa83
85d45a0
1e1cccf
da93f7f
01f4045
7828453
c7f11c4
a6acb0b
e85bfe1
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,5 @@ | ||
| [bumpversion] | ||
| current_version = 0.8.18 | ||
| current_version = 0.9.0 | ||
| commit = True | ||
| tag = True | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,46 +1,31 @@ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| # .github/workflows/deploy.yml | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| name: Deploy 🚀 Clima to Google Cloud Run (☁🏃) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| on: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| push: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| branches: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - main | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| jobs: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| deploy: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| name: Deploying 🚀 Clima | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| runs-on: ubuntu-latest | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| if: "contains(github.event.head_commit.message, 'bump version')" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| steps: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Checkout | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| uses: actions/checkout@v2 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| uses: actions/checkout@v4 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Setup python | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| uses: actions/setup-python@v4 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - id: auth | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| uses: google-github-actions/auth@v2 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| python-version: '3.11' | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Export gcloud related env variable | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| run: export CLOUDSDK_PYTHON="/usr/bin/python3" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| credentials_json: ${{ secrets.GCP_SA_KEY_JSON }} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| # Build and push image to Google Container Registry | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Setting up | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| uses: google-github-actions/setup-gcloud@v0 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Set up Cloud SDK | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| uses: google-github-actions/setup-gcloud@v1 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. setup-gcloud@v1 is deprecated on new runners; bump to v2 actionlint warns v1 is too old. Upgrade to v2. - - name: Set up Cloud SDK
- uses: google-github-actions/setup-gcloud@v1
+ - name: Set up Cloud SDK
+ uses: google-github-actions/setup-gcloud@v2🧰 Tools🪛 actionlint (1.7.7)23-23: the runner of "google-github-actions/setup-gcloud@v1" action is too old to run on GitHub Actions. update the action's version to fix this issue (action) 🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| version: '318.0.0' | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| service_account_key: ${{ secrets.GCP_SA_KEY_JSON }} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| service_account_email: "federico.tartarini@bears-berkeley.sg" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| project_id: clima-316917 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| project_id: heat-stress-scale | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Building (🏗️) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| run: |- | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Building (🏗️) and Deploying (🚀) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| gcloud builds submit \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --tag us-docker.pkg.dev/clima-316917/gcr.io/clima | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| # Setup gcloud CLI | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Deploy (🚀) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| uses: google-github-actions/deploy-cloudrun@v1 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| service: clima | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| image: us-docker.pkg.dev/clima-316917/gcr.io/clima | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| region: us-central1 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| credentials: ${{ secrets.GCP_SA_KEY_JSON }} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| project_id: clima-316917 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --project=clima-316917 \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --substitutions=_REPO_NAME="clima",_PROJ_NAME="clima-316917",_IMG_NAME="main",_GCR="us.gcr.io",_REGION="us-central1",_MEMORY="4Gi",_CPU="2" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
25
to
31
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Project mismatch: setup uses heat-stress-scale, build submits to clima-316917 These should match; otherwise auth/quotas/logs land in different projects. Align both to the intended GCP project (pick one). - with:
- project_id: heat-stress-scale
+ with:
+ project_id: clima-316917
@@
- --project=clima-316917 \
+ --project=${{ steps.gcloud_setup.outputs.project_id || 'clima-316917' }} \Or hardcode one project consistently in both places. 📝 Committable suggestion
Suggested change
🧰 Tools🪛 YAMLlint (1.37.1)[error] 31-31: no new line character at the end of file (new-line-at-end-of-file) 🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,10 @@ | ||||||||||||||||||||||||||||||||
| repos: | ||||||||||||||||||||||||||||||||
| - repo: https://github.com/astral-sh/ruff-pre-commit | ||||||||||||||||||||||||||||||||
| # Ruff version. | ||||||||||||||||||||||||||||||||
| rev: v0.12.9 | ||||||||||||||||||||||||||||||||
| hooks: | ||||||||||||||||||||||||||||||||
| # Run the linter. | ||||||||||||||||||||||||||||||||
| - id: ruff-check | ||||||||||||||||||||||||||||||||
| args: [ --fix ] | ||||||||||||||||||||||||||||||||
| # Run the formatter. | ||||||||||||||||||||||||||||||||
| - id: ruff-format | ||||||||||||||||||||||||||||||||
|
Comment on lines
+7
to
+10
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛠️ Refactor suggestion Fix YAML indentation under hooks and add missing EOF newline Current indentation under hooks is deeper than pre-commit’s canonical style and flagged by YAMLlint; also there’s no trailing newline. repos:
- repo: https://github.com/astral-sh/ruff-pre-commit
# Ruff version.
rev: v0.12.9
hooks:
- # Run the linter.
- - id: ruff-check
- args: [ --fix ]
- # Run the formatter.
- - id: ruff-format
+ # Run the linter.
+ - id: ruff-check
+ args: [--fix]
+ # Run the formatter.
+ - id: ruff-format
+📝 Committable suggestion
Suggested change
🧰 Tools🪛 YAMLlint (1.37.1)[warning] 7-7: wrong indentation: expected 2 but found 4 (indentation) [error] 10-10: no new line character at the end of file (new-line-at-end-of-file) 🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -23,6 +23,7 @@ pytest = "*" | |
| bump2version = "*" | ||
| black = "*" | ||
| ruff = "*" | ||
| pre-commit = "*" | ||
|
|
||
| [requires] | ||
| python_version = "3.11" | ||
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -5,11 +5,72 @@ description: Guide on how to contribute to this project | |||||||||||||||||||
| # How to contribute | ||||||||||||||||||||
|
|
||||||||||||||||||||
| First off, thanks for taking the time to contribute! | ||||||||||||||||||||
| We use GitHub as our main collaboration platform. Please work from the `development` branch, create small feature branches, and open focused pull requests. Follow Conventional Commit messages (e.g., `feat:`, `fix:`, `docs:`), format Python code with Black, and add tests where needed. Never merge your own PR—wait for review and address all comments (including AI reviewer suggestions). Use Issues and Projects to track tasks and discussions. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| > This project requires Python 3.11. Do not use Python 3.12 or newer, as it may cause dependency incompatibilities, build failure or runtime errors | ||||||||||||||||||||
|
|
||||||||||||||||||||
|
Comment on lines
+8
to
+11
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛠️ Refactor suggestion Align formatter guidance: choose Ruff or Black (docs vs CI mismatch) CI uses Ruff (check/format). The doc says “format Python code with Black” and later installs Black. This is conflicting. Recommend standardizing on Ruff (formatter) and dropping Black from mandatory steps. -We use GitHub as our main collaboration platform. Please work from the `development` branch, create small feature branches, and open focused pull requests. Follow Conventional Commit messages (e.g., `feat:`, `fix:`, `docs:`), format Python code with Black, and add tests where needed. Never merge your own PR—wait for review and address all comments (including AI reviewer suggestions). Use Issues and Projects to track tasks and discussions.
+We use GitHub as our main collaboration platform. Please work from the `development` branch, create small feature branches, and open focused pull requests. Follow Conventional Commits (e.g., `feat:`, `fix:`, `docs:`), format Python code with Ruff (`ruff format`), and add tests where needed. Never merge your own PR—wait for review and address all comments (including AI reviewer suggestions). Use Issues and Projects to track tasks and discussions.📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||
|
|
||||||||||||||||||||
| ## General Feedback | ||||||||||||||||||||
|
|
||||||||||||||||||||
| If you have a general feedback about our project, please do not open an issue but instead please fill in this [form](https://forms.gle/LRUq3vsFnE1QCLiA6) | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ## Fork & branch processing | ||||||||||||||||||||
|
|
||||||||||||||||||||
| First fork the origin repository to your own github repository, then clone the repository to your local computer. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ```bash | ||||||||||||||||||||
| git clone https://github.com/Your Account name/clima.git | ||||||||||||||||||||
| cd clima | ||||||||||||||||||||
| ``` | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Set up the upstream repository and check the output repositories. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ```bash | ||||||||||||||||||||
| git remote add upstream https://github.com/CenterForTheBuiltEnvironment/clima.git | ||||||||||||||||||||
|
|
||||||||||||||||||||
| git remote -v | ||||||||||||||||||||
| ``` | ||||||||||||||||||||
|
|
||||||||||||||||||||
| The terminal should output a list: | ||||||||||||||||||||
|
|
||||||||||||||||||||
| - `origin → your Fork repository` | ||||||||||||||||||||
| - `upstream → origin repository` | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Check all branches. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ```bash | ||||||||||||||||||||
| git branch -a | ||||||||||||||||||||
| ``` | ||||||||||||||||||||
|
|
||||||||||||||||||||
| The terminal will show a list of branches: | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ```bash | ||||||||||||||||||||
| > * main | ||||||||||||||||||||
| remotes/origin/HEAD -> origin/main | ||||||||||||||||||||
| remotes/origin/development | ||||||||||||||||||||
| remotes/origin/main | ||||||||||||||||||||
| ``` | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Pull the development branch first, and if the terminal does not notice you that you should try the second command. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ```bash | ||||||||||||||||||||
| git checktout development | ||||||||||||||||||||
|
|
||||||||||||||||||||
| git checkout -b development origin/development | ||||||||||||||||||||
| ``` | ||||||||||||||||||||
|
|
||||||||||||||||||||
|
Comment on lines
+54
to
+61
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Typos and phrasing in branch checkout instructions Fix “checktout” and clarify the fallback command. -Pull the development branch first, and if the terminal does not notice you that you should try the second command.
+Pull the development branch. If that fails, try the second command to create a local tracking branch.
@@
-git checktout development
+git checkout development📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||
| Create a new branch in the development branch. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ```bash | ||||||||||||||||||||
| git checkout -b (your branch name) | ||||||||||||||||||||
| ``` | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Finally update and push to your repository branch if you modify the files. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ```bash | ||||||||||||||||||||
| git push origin (your branch name) | ||||||||||||||||||||
| ``` | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ## Code of Conduct | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Available [here](code_of_conduct.md) | ||||||||||||||||||||
|
|
@@ -18,6 +79,57 @@ Available [here](code_of_conduct.md) | |||||||||||||||||||
|
|
||||||||||||||||||||
| We use Black.exe to format the code. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Install Black: | ||||||||||||||||||||
|
|
||||||||||||||||||||
| We use ruff to enforce the code style and code formatting. You can run it with: | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ```bash | ||||||||||||||||||||
| pipenv run ruff check . | ||||||||||||||||||||
| pipenv run ruff format . | ||||||||||||||||||||
| ``` | ||||||||||||||||||||
|
|
||||||||||||||||||||
| To ensure that the code is formatted correctly, we use a pre-commit hook that runs Ruff before every commit. | ||||||||||||||||||||
| Run the following once to enable hooks in your local repo: | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ```bash | ||||||||||||||||||||
| pipenv run pre-commit install | ||||||||||||||||||||
| # optional: run on all files | ||||||||||||||||||||
| pipenv run pre-commit run --all-files | ||||||||||||||||||||
| ``` | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Hence, you will need to make sure that the code is formatted correctly before committing your changes; otherwise, the commit will fail. | ||||||||||||||||||||
| More information about pre-commit hooks can be found [here](https://pre-commit.com/). | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ```bash | ||||||||||||||||||||
| pipenv install black | ||||||||||||||||||||
| ``` | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Format your code before committing: | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ```bash | ||||||||||||||||||||
| black . | ||||||||||||||||||||
| ``` | ||||||||||||||||||||
|
|
||||||||||||||||||||
|
||||||||||||||||||||
| ## Testing | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Before submitting a Pull Request, please make sure: | ||||||||||||||||||||
| - All tests should pass. | ||||||||||||||||||||
| - You have installed project dependencies: | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ```bash | ||||||||||||||||||||
| npm install | ||||||||||||||||||||
|
|
||||||||||||||||||||
| pipenv install -r requirements.txt | ||||||||||||||||||||
| ``` | ||||||||||||||||||||
|
|
||||||||||||||||||||
| From the root directory, run: | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ```bash | ||||||||||||||||||||
| cd tests/node | ||||||||||||||||||||
|
|
||||||||||||||||||||
| npx cypress run | ||||||||||||||||||||
| ``` | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ## Submitting changes | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Please send a Pull Request with a clear list of what you've done. Always write a clear log message for your commits. One-line messages are fine for small changes, but bigger changes should look like this: | ||||||||||||||||||||
|
|
@@ -28,6 +140,43 @@ $ git commit -m "A brief summary of the commit | |||||||||||||||||||
| > A paragraph describing what changed and its impact." | ||||||||||||||||||||
| ``` | ||||||||||||||||||||
|
|
||||||||||||||||||||
| > Detailed requirements for submitting a PR are described in the [Pull Request Regulation](#pull-request-regulation) section below | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Classification of Common Commit Types: | ||||||||||||||||||||
|
|
||||||||||||||||||||
| - `Main (Master)`: Stable branch, merge code that passes review and CI; merge and release every time, | ||||||||||||||||||||
| - `Develop`: Continuous Integration branch for daily integration with multiple collaborators. | ||||||||||||||||||||
| - `Feature/*`: feature development branch, cut out from main or develop, send PR to merge in after completing the feature. | ||||||||||||||||||||
| - `Fix/*`: defect repair branch, the same process as feature | ||||||||||||||||||||
| - `Release/*`: release preparation branch for freezing versions, fixing documentation, doing regressions and tagging. | ||||||||||||||||||||
| - `docs/*`, `chore/*`, `refactor/*`, `test/*`: documentation, miscellaneous, refactor, test type branches. | ||||||||||||||||||||
| - `Style`: style modification (does not affect the function): code formatting, space adjustment, naming rules unity. | ||||||||||||||||||||
| - `Refactor`: Code Refactoring: Refactor existing code to improve maintainability. | ||||||||||||||||||||
| - `Test`: Add or modify tests: add unit tests, integration tests, or modify test logic. | ||||||||||||||||||||
| - `Chore`: Build Configuration, Dependency Management, CI/CD Configuration Updates. | ||||||||||||||||||||
| - `Perf`: Performance Optimisation: Optimising code execution efficiency or memory usage. | ||||||||||||||||||||
| - `Ci`: CI Configuration Related: Changing Continuous Integration Configurations for Github Actions, Travis, Jenkins, etc. | ||||||||||||||||||||
| - `Build`: build system related: modify build scripts, packaging configuration. | ||||||||||||||||||||
| - `Revert`: Rollback Commit: Undoing a Previous Commit | ||||||||||||||||||||
| - `Security`: Security fixes, fixing security vulnerabilities, updating dependencies to prevent attacks. | ||||||||||||||||||||
| - `Deps`: Dependency Management: Dependency Management/Adding, updating, and removing dependency libraries | ||||||||||||||||||||
| - `Infra`: Infrastructure related: changes to development environments, containers, server configurations, etc. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| ## Pull Request Regulation | ||||||||||||||||||||
| **Time to submit PR:** | ||||||||||||||||||||
|
|
||||||||||||||||||||
| - User requirements/issues have been addressed or discussed in Issue and consensus has been reached. | ||||||||||||||||||||
| - Changes have been minimised (small steps/phased submission) to avoid "mega PRs". | ||||||||||||||||||||
|
|
||||||||||||||||||||
| **The pull request should include the following information:** | ||||||||||||||||||||
|
|
||||||||||||||||||||
| - **Description:** Provide a brief summary of the changes, related issues, and motivation. List any required dependencies. **Fixes # (issue)** | ||||||||||||||||||||
|
|
||||||||||||||||||||
| - **Type of Change:** Bug fix (non-breaking); New feature (non-breaking); Breaking change; Documentation update. | ||||||||||||||||||||
|
|
||||||||||||||||||||
| - **Testing:** Describe how you tested your changes and how we can reproduce them. Include test details if necessary. | ||||||||||||||||||||
|
|
||||||||||||||||||||
|
|
||||||||||||||||||||
| ## Thanks | ||||||||||||||||||||
|
|
||||||||||||||||||||
| Thank you again for being interested in this project! You are awesome! | ||||||||||||||||||||
|
|
||||||||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Verification agent
🧩 Analysis chain
CI won’t run on PRs to main with these triggers.
This PR targets main, but the workflow triggers only on development. End-to-end tests will be skipped for main PRs and pushes to main.
Please re-run this PR after the change to confirm the Cypress job triggers on main.
🏁 Script executed:
Length of output: 561
Add “main” to the Cypress workflow triggers
The current workflow only runs on pushes and PRs against development, so any PR targeting main (like this one) will skip end-to-end tests. Please update
.github/workflows/cypress.ymlto include both main and development—and add a manual trigger if desired—so CI runs on main as well.• File:
.github/workflows/cypress.yml• Replace lines 2–8 with:
After making this change, re-run the workflow on this PR to confirm that the Cypress job now triggers for main branch changes.
📝 Committable suggestion
🤖 Prompt for AI Agents