-
Notifications
You must be signed in to change notification settings - Fork 348
Release v2 of Resonate App with all recent changes #537
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
6dd4d40
7edc689
48f81f8
2a8bd56
dfd6ff6
a2b78f9
6c497fa
a1bd21a
6167816
6be5cd5
1d1ef6e
1afcfba
15794c1
d99aa60
7fad0ce
c9a9168
ed06fe6
c0d16ff
ad5ea52
5fd7b42
99d7bcb
a2a2724
0bb6bd7
454247a
1964623
1bc1cc0
d411ed0
02a1c54
92cd416
731122c
39699f6
d0ad85a
5a5256b
8d3769e
fa8c3a6
f2929b0
b971f5a
9028057
9475443
7f41f38
73840f8
8137500
09a049f
703deb0
04579f2
b355d5d
6c8a772
2d70f81
9b92027
2a8e848
7a9d012
ce09158
f2f9ed5
f490116
dffd8cd
6664d86
280cbc7
ae379c8
4a2ce8b
de0a024
2b0ce43
1fa4efa
483646b
cf6bc08
7bc7f65
6578510
0dfe135
c8c7905
9886d69
1df787a
22b0996
82b1620
0f048aa
8de7aaa
77562b7
6794295
aab7394
50d6883
413132c
29af01e
1e57bf7
241dffa
6784c70
c2208b7
e2183fa
78f6abc
54c0f26
8fab528
bd62f0e
1ad704b
57d436b
bc90258
210b474
1e5d2f6
9d3ce64
9cfad28
596bf6c
fab8da4
312be5e
7114b91
87e5189
83b42a6
eac8b75
636d39e
bd26eee
a477446
eebfead
de97cff
188a735
d525cff
0ec9cf1
d8b29a5
cef354c
228cfec
ed0de10
e81fad3
c2c7326
5785984
461766e
b025fc3
499af73
c726a71
28e4893
37a48bb
b67d193
ca2a9be
cd03350
bbf022a
d3d17bd
cba2b74
424a1f5
eaaa4ce
6df2345
62de758
a25ea31
d0a7fe9
3025c13
ddbf872
0d36952
00e5a5d
67d9849
d529a8f
b24d901
5072224
a885c22
2dab644
882b8dc
ad51d1f
0662f5a
6f97a60
37d5f53
4bd3198
ce4078c
63a0a6c
e85d2f2
251e8e7
b547404
1dbed55
956632a
bae0f97
208f68e
dbd2ad3
7d82a49
c447236
df48948
b6e8672
5f07656
2acdc8d
9766bfc
236a4d6
4a0ab02
efd8712
49145ce
17b8a30
ce7dfa6
120960b
a852e39
ab5abfa
52ba163
205c26f
669f64e
fcae591
3595bf7
d26c5f1
a46316f
2b6aee8
ad95a50
5c7942b
59d2417
365924b
ef39d7c
3609d13
92ab1e3
16ad5eb
9807bbf
330249b
83bb7f6
c289122
8152ea7
688b0fa
e1bbc38
5e62bbe
2699cf7
eb2de02
ac21564
867c3d3
b2418e6
104e650
bf39ab1
d73c017
04a36ee
a7a10d1
53300b7
f653417
3ea7c87
671922d
bfde40d
e01eaec
45e1577
d6e3864
f8bbd54
bfc4fb7
767c586
6bbeece
259edfc
7dcf1c7
fb3e5c3
c6fd695
38d66d0
aacd078
8faaf0d
7111ce9
f1aac3f
f7c42a1
8ad63b0
c513074
dbbda64
2e63eb7
0469078
0dc9b57
fe3772e
7624801
5997eb7
18a3b55
4f11c4f
30aaba0
1e7d643
225f23d
6ed83ab
494e305
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 |
|---|---|---|
| @@ -0,0 +1,39 @@ | ||
| name: ⚙️ Configuration Change | ||
| description: Suggest modifications to the project’s configuration, such as build tools, CI/CD pipeline adjustments, or environment settings to improve project management or deployment. | ||
|
|
||
| body: | ||
| - type: textarea | ||
| id: configuration-change | ||
| attributes: | ||
| label: ⚙️ The Configuration Change | ||
| description: > | ||
| Describe the specific configuration change you propose. What configuration settings or tools need adjustment? | ||
| Mention which part of the project this affects, like build tools, CI/CD settings, or deployment configurations. | ||
| validations: | ||
| required: true | ||
|
|
||
| - type: textarea | ||
| attributes: | ||
| label: Motivation, pitch | ||
| description: > | ||
| Explain the reason for this configuration change. How will it improve the project's management or deployment process? | ||
| For example, will it streamline the CI/CD pipeline, improve build times, or make the project easier to maintain? | ||
| validations: | ||
| required: true | ||
|
|
||
| - type: checkboxes | ||
| attributes: | ||
| label: Configuration Change Focus Areas | ||
| options: | ||
| - "Build tools" | ||
| - "CI/CD pipeline" | ||
| - "Environment settings" | ||
| - "Dependency management" | ||
| - "Other" | ||
| validations: | ||
| required: false | ||
|
|
||
| - type: markdown | ||
| attributes: | ||
| value: > | ||
| Thank you for your contribution! 🔧 Please provide as much information as possible about the configuration change and how it will benefit the project. |
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,40 @@ | ||||||||||||||||||||||||||||||||||||||||||||||||
| name: 🔄 Re-Implementation | ||||||||||||||||||||||||||||||||||||||||||||||||
| description: Suggest rewriting or replacing a part of the codebase for better performance, maintainability, or functionality. | ||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||
| body: | ||||||||||||||||||||||||||||||||||||||||||||||||
| - type: textarea | ||||||||||||||||||||||||||||||||||||||||||||||||
| id: re-implementation | ||||||||||||||||||||||||||||||||||||||||||||||||
| attributes: | ||||||||||||||||||||||||||||||||||||||||||||||||
| label: 🔄 The re-implementation | ||||||||||||||||||||||||||||||||||||||||||||||||
| description: > | ||||||||||||||||||||||||||||||||||||||||||||||||
| A clear and concise description of the part of the system that you think needs re-implementation. | ||||||||||||||||||||||||||||||||||||||||||||||||
| What specific functionality or code block would you like to rewrite or replace? Please explain the issue in detail. | ||||||||||||||||||||||||||||||||||||||||||||||||
| validations: | ||||||||||||||||||||||||||||||||||||||||||||||||
| required: true | ||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||
| - type: textarea | ||||||||||||||||||||||||||||||||||||||||||||||||
| attributes: | ||||||||||||||||||||||||||||||||||||||||||||||||
| label: Motivation, pitch | ||||||||||||||||||||||||||||||||||||||||||||||||
| description: > | ||||||||||||||||||||||||||||||||||||||||||||||||
| Please provide the motivation for re-implementing this part of the code. | ||||||||||||||||||||||||||||||||||||||||||||||||
| Why do you think the current implementation needs to be replaced or rewritten? | ||||||||||||||||||||||||||||||||||||||||||||||||
| E.g., *"The current implementation is slow, hard to maintain, or outdated."* If you have any supporting context or examples, please include them here. | ||||||||||||||||||||||||||||||||||||||||||||||||
| validations: | ||||||||||||||||||||||||||||||||||||||||||||||||
| required: true | ||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||
| - type: checkboxes | ||||||||||||||||||||||||||||||||||||||||||||||||
| attributes: | ||||||||||||||||||||||||||||||||||||||||||||||||
| label: Areas of Focus | ||||||||||||||||||||||||||||||||||||||||||||||||
| options: | ||||||||||||||||||||||||||||||||||||||||||||||||
| - "Performance" | ||||||||||||||||||||||||||||||||||||||||||||||||
| - "Maintainability" | ||||||||||||||||||||||||||||||||||||||||||||||||
| - "Code readability" | ||||||||||||||||||||||||||||||||||||||||||||||||
| - "Functionality improvement" | ||||||||||||||||||||||||||||||||||||||||||||||||
| - "Other" | ||||||||||||||||||||||||||||||||||||||||||||||||
| validations: | ||||||||||||||||||||||||||||||||||||||||||||||||
| required: false | ||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+25
to
+35
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. Checkboxes block missing id Add an - - type: checkboxes
- attributes:
+ - type: checkboxes
+ id: focus-areas
+ attributes:
label: Areas of Focus📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||
| - type: markdown | ||||||||||||||||||||||||||||||||||||||||||||||||
| attributes: | ||||||||||||||||||||||||||||||||||||||||||||||||
| value: > | ||||||||||||||||||||||||||||||||||||||||||||||||
| Thanks for contributing 🎉! Be as detailed as you can in your suggestion. The more information you provide, the easier it will be to understand the need for re-implementation and how it can improve the project! | ||||||||||||||||||||||||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,40 @@ | ||||||||||||||||||||||||||||||||||||||||
| name: 🔧 Refactor to Enhance | ||||||||||||||||||||||||||||||||||||||||
| description: Suggest a refactor to improve the code's efficiency, readability, or maintainability while keeping its functionality intact. | ||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
| body: | ||||||||||||||||||||||||||||||||||||||||
| - type: textarea | ||||||||||||||||||||||||||||||||||||||||
| id: refactor-to-enhance | ||||||||||||||||||||||||||||||||||||||||
| attributes: | ||||||||||||||||||||||||||||||||||||||||
| label: 🔧 The refactor | ||||||||||||||||||||||||||||||||||||||||
| description: > | ||||||||||||||||||||||||||||||||||||||||
| Describe the part of the code that you think needs refactoring. What is the current problem or area that could benefit from enhancement? | ||||||||||||||||||||||||||||||||||||||||
| How do you think refactoring will improve the codebase? | ||||||||||||||||||||||||||||||||||||||||
| validations: | ||||||||||||||||||||||||||||||||||||||||
| required: true | ||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
| - type: textarea | ||||||||||||||||||||||||||||||||||||||||
| attributes: | ||||||||||||||||||||||||||||||||||||||||
| label: Motivation, pitch | ||||||||||||||||||||||||||||||||||||||||
| description: > | ||||||||||||||||||||||||||||||||||||||||
| Please explain the motivation for this refactor. Why is this refactor needed? | ||||||||||||||||||||||||||||||||||||||||
| What benefits will it bring to the codebase, such as improved performance, easier maintenance, or better readability? | ||||||||||||||||||||||||||||||||||||||||
| Provide examples of how the code can be improved if possible. | ||||||||||||||||||||||||||||||||||||||||
| validations: | ||||||||||||||||||||||||||||||||||||||||
| required: true | ||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+15
to
+24
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. Missing id for the second textarea Add a unique - - type: textarea
- attributes:
+ - type: textarea
+ id: motivation
+ attributes:
label: Motivation, pitch📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||
| - type: checkboxes | ||||||||||||||||||||||||||||||||||||||||
| attributes: | ||||||||||||||||||||||||||||||||||||||||
| label: Refactoring Focus Areas | ||||||||||||||||||||||||||||||||||||||||
| options: | ||||||||||||||||||||||||||||||||||||||||
| - "Performance" | ||||||||||||||||||||||||||||||||||||||||
| - "Code readability" | ||||||||||||||||||||||||||||||||||||||||
| - "Maintainability" | ||||||||||||||||||||||||||||||||||||||||
| - "Reducing complexity" | ||||||||||||||||||||||||||||||||||||||||
| - "Other" | ||||||||||||||||||||||||||||||||||||||||
| validations: | ||||||||||||||||||||||||||||||||||||||||
| required: false | ||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+25
to
+35
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. Add id to checkboxes block Without an - - type: checkboxes
- attributes:
+ - type: checkboxes
+ id: focus-areas
+ attributes:
label: Refactoring Focus Areas🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||
| - type: markdown | ||||||||||||||||||||||||||||||||||||||||
| attributes: | ||||||||||||||||||||||||||||||||||||||||
| value: > | ||||||||||||||||||||||||||||||||||||||||
| Thank you for suggesting a refactor! 🎉 Be sure to provide as much detail as possible, and help the team understand how the refactor can improve the codebase! | ||||||||||||||||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,39 @@ | ||||||||||||||||||||||||||||||||||||
| name: 🔒 Security Improvement | ||||||||||||||||||||||||||||||||||||
| description: Suggest changes to improve the project's security, including identifying vulnerabilities and proposing measures to harden the codebase. | ||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||
| body: | ||||||||||||||||||||||||||||||||||||
| - type: textarea | ||||||||||||||||||||||||||||||||||||
| id: security-improvement | ||||||||||||||||||||||||||||||||||||
| attributes: | ||||||||||||||||||||||||||||||||||||
| label: 🔒 The Security Improvement | ||||||||||||||||||||||||||||||||||||
| description: > | ||||||||||||||||||||||||||||||||||||
| Describe the security improvement you propose. Identify any potential risks or vulnerabilities in the current setup | ||||||||||||||||||||||||||||||||||||
| and detail the changes required to enhance security. Specify whether it impacts data handling, user authentication, or any external dependencies. | ||||||||||||||||||||||||||||||||||||
| validations: | ||||||||||||||||||||||||||||||||||||
| required: true | ||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||
| - type: textarea | ||||||||||||||||||||||||||||||||||||
| attributes: | ||||||||||||||||||||||||||||||||||||
| label: Motivation, pitch | ||||||||||||||||||||||||||||||||||||
| description: > | ||||||||||||||||||||||||||||||||||||
| Explain the motivation behind this security improvement. How does it protect against security risks, and what kind of | ||||||||||||||||||||||||||||||||||||
| vulnerabilities does it address? Describe how it can prevent unauthorized access, data breaches, or other threats. | ||||||||||||||||||||||||||||||||||||
| validations: | ||||||||||||||||||||||||||||||||||||
| required: true | ||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||
|
Comment on lines
+15
to
+23
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. Missing id for textarea breaks GitHub issue form Each input needs a unique Apply: - - type: textarea
- attributes:
+ - type: textarea
+ id: motivation
+ attributes:
label: Motivation, pitch📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||
| - type: checkboxes | ||||||||||||||||||||||||||||||||||||
| attributes: | ||||||||||||||||||||||||||||||||||||
| label: Security Improvement Focus Areas | ||||||||||||||||||||||||||||||||||||
| options: | ||||||||||||||||||||||||||||||||||||
| - "Data protection" | ||||||||||||||||||||||||||||||||||||
| - "User authentication" | ||||||||||||||||||||||||||||||||||||
| - "Dependency vulnerabilities" | ||||||||||||||||||||||||||||||||||||
| - "Access control" | ||||||||||||||||||||||||||||||||||||
| - "Other" | ||||||||||||||||||||||||||||||||||||
| validations: | ||||||||||||||||||||||||||||||||||||
| required: false | ||||||||||||||||||||||||||||||||||||
|
Comment on lines
+24
to
+34
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. Add id to the checkboxes block
Apply: - - type: checkboxes
- attributes:
+ - type: checkboxes
+ id: focus-areas
+ attributes:
label: Security Improvement Focus Areas🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||
| - type: markdown | ||||||||||||||||||||||||||||||||||||
| attributes: | ||||||||||||||||||||||||||||||||||||
| value: > | ||||||||||||||||||||||||||||||||||||
| Thank you for prioritizing security! 🔒 Be as thorough as possible in identifying potential risks and detailing how your proposal will safeguard the project. | ||||||||||||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,82 @@ | ||||||||||||||||||||||||||||
| name: Build and Deploy | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| on: | ||||||||||||||||||||||||||||
| push: | ||||||||||||||||||||||||||||
| branches: | ||||||||||||||||||||||||||||
| - master | ||||||||||||||||||||||||||||
| pull_request: | ||||||||||||||||||||||||||||
| types: [closed] | ||||||||||||||||||||||||||||
| branches: | ||||||||||||||||||||||||||||
| - master | ||||||||||||||||||||||||||||
| workflow_dispatch: | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| jobs: | ||||||||||||||||||||||||||||
| build: | ||||||||||||||||||||||||||||
| runs-on: ubuntu-latest | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
|
Comment on lines
+14
to
+16
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. Gate runs on closed PRs to only execute when merged. As written, closing an unmerged PR will still run both jobs. Add job-level guards. jobs:
build:
+ if: ${{ github.event_name != 'pull_request' || github.event.pull_request.merged == true }}
runs-on: ubuntu-latest
@@
release:
+ if: ${{ github.event_name != 'pull_request' || github.event.pull_request.merged == true }}
runs-on: ubuntu-latestAlso applies to: 57-62 🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||
| steps: | ||||||||||||||||||||||||||||
| # 1. Remove Cached Flutter Installation | ||||||||||||||||||||||||||||
| - name: Remove Cached Flutter Installation | ||||||||||||||||||||||||||||
| run: rm -rf $HOME/.flutter | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| - name: Clean up old build files | ||||||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||||||
| rm -rf build/app/outputs/bundle/release/* | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| # 2. Checkout repository | ||||||||||||||||||||||||||||
| - name: Checkout code | ||||||||||||||||||||||||||||
| uses: actions/checkout@v4 | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| # 3. Install Flutter SDK | ||||||||||||||||||||||||||||
| - name: Install Flutter | ||||||||||||||||||||||||||||
| uses: subosito/flutter-action@v2 | ||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||
| flutter-version: 3.35.2 | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
|
Comment on lines
+30
to
+35
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. Ensure Java 17 for modern AGP/Android builds. Set up JDK 17 explicitly; relying on runner defaults is brittle. - name: Install Flutter
uses: subosito/flutter-action@v2
with:
flutter-version: 3.35.2
+ channel: stable
+ cache: true
+
+ - name: Set up Java 17
+ uses: actions/setup-java@v4
+ with:
+ distribution: 'temurin'
+ java-version: '17'Also applies to: 36-41 🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||
| # 4. Verify Flutter and Dart Versions | ||||||||||||||||||||||||||||
| - name: Verify Flutter Installation | ||||||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||||||
| flutter --version | ||||||||||||||||||||||||||||
| dart --version | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| # 6. Install Flutter dependencies | ||||||||||||||||||||||||||||
| - name: Install Flutter Dependencies | ||||||||||||||||||||||||||||
| run: flutter pub get | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| # 7. Build Android APK | ||||||||||||||||||||||||||||
| - name: Build Android App Bundle | ||||||||||||||||||||||||||||
| run: flutter build apk --release --split-per-abi | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| # 8. Upload build artifacts | ||||||||||||||||||||||||||||
| - name: Upload APK as an artifact | ||||||||||||||||||||||||||||
| uses: actions/upload-artifact@v4 | ||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||
| name: app-release-apks | ||||||||||||||||||||||||||||
| path: build/app/outputs/apk/release/*.apk | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| release: | ||||||||||||||||||||||||||||
| runs-on: ubuntu-latest | ||||||||||||||||||||||||||||
| needs: build | ||||||||||||||||||||||||||||
| permissions: | ||||||||||||||||||||||||||||
| contents: write | ||||||||||||||||||||||||||||
| steps: | ||||||||||||||||||||||||||||
|
Comment on lines
+57
to
+62
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. Gate releases on merged PRs only.
release:
runs-on: ubuntu-latest
needs: build
permissions:
contents: write
+ if: ${{ github.event_name != 'pull_request' || github.event.pull_request.merged == true }}📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||
| # 1. Checkout repository | ||||||||||||||||||||||||||||
| - name: Checkout code | ||||||||||||||||||||||||||||
| uses: actions/checkout@v4 | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| # 2. Download build artifacts | ||||||||||||||||||||||||||||
| - name: Download APK | ||||||||||||||||||||||||||||
| uses: actions/download-artifact@v4 | ||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||
| name: app-release-apks | ||||||||||||||||||||||||||||
| path: build/app/outputs/bundle/release/ | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| # 3. Upload to Github Release | ||||||||||||||||||||||||||||
| - name: Github Release | ||||||||||||||||||||||||||||
| uses: ncipollo/[email protected] | ||||||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||||||
| allowUpdates: true | ||||||||||||||||||||||||||||
| artifacts: build/app/outputs/bundle/release/*.apk | ||||||||||||||||||||||||||||
| artifactContentType: apk | ||||||||||||||||||||||||||||
| generateReleaseNotes: true | ||||||||||||||||||||||||||||
| tag: latest_build | ||||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,9 +2,9 @@ name: Greet Contributors | |
|
|
||
| on: | ||
| issues: | ||
| types: [opened] | ||
| pull_request: | ||
| types: [opened] | ||
| types: [opened, closed] | ||
| pull_request_target: | ||
| types: [opened, closed] | ||
|
Comment on lines
+6
to
+7
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. Use pull_request instead of pull_request_target for greetings. No need for base-repo token elevation here. Switching to on:
issues:
types: [opened, closed]
- pull_request_target:
+ pull_request:
types: [opened, closed]
@@
- - name: Handle Opened PRs
- if: github.event_name == 'pull_request_target' && github.event.action == 'opened'
+ - name: Handle Opened PRs
+ if: github.event_name == 'pull_request' && github.event.action == 'opened'
@@
- - name: Handle Closed PRs
- if: github.event_name == 'pull_request_target' && github.event.action == 'closed'
+ - name: Handle Closed PRs
+ if: github.event_name == 'pull_request' && github.event.action == 'closed'Also applies to: 50-51, 66-68 |
||
|
|
||
| permissions: | ||
| issues: write | ||
|
|
@@ -16,34 +16,66 @@ jobs: | |
| runs-on: ubuntu-latest | ||
|
|
||
| steps: | ||
| - name: Print GitHub context | ||
| run: echo "${{ toJson(github) }}" | ||
| - name: Handle Opened Issues | ||
| if: github.event_name == 'issues' && github.event.action == 'opened' | ||
| uses: peter-evans/create-or-update-comment@v4 | ||
| with: | ||
| token: ${{ secrets.GITHUB_TOKEN }} | ||
| issue-number: ${{ github.event.issue.number }} | ||
| body: | | ||
| 👋 **Hello @${{ github.actor }}!** | ||
| Thank you for opening this issue. Our team will review it soon. 🚀 | ||
| - If you can, please provide more details like steps to reproduce, expected vs. actual behavior, and screenshots (if applicable). 📌 | ||
|
|
||
| We appreciate your contribution! 💡 | ||
| reactions: heart | ||
|
|
||
| - name: Handle Closed Issues | ||
| if: github.event_name == 'issues' && github.event.action == 'closed' | ||
| uses: peter-evans/create-or-update-comment@v4 | ||
| with: | ||
| token: ${{ secrets.GITHUB_TOKEN }} | ||
| issue-number: ${{ github.event.issue.number }} | ||
| body: | | ||
| 🔒 **Issue Closed - Thank You, @${{ github.actor }}!** | ||
| This issue has been resolved or is no longer relevant. | ||
| - If you have further questions, feel free to open a new issue. 💡 | ||
|
|
||
| - name: Send greeting message for issues | ||
| if: github.event_name == 'issues' | ||
| uses: actions-ecosystem/[email protected] | ||
| continue-on-error: true | ||
| Thanks for helping improve the project! 🚀 | ||
| reactions: | | ||
| heart | ||
| hooray | ||
|
|
||
| - name: Handle Opened PRs | ||
| if: github.event_name == 'pull_request_target' && github.event.action == 'opened' | ||
| uses: peter-evans/create-or-update-comment@v4 | ||
| with: | ||
| github_token: ${{ secrets.GITHUB_TOKEN }} | ||
| repo: ${{ github.repository }} | ||
| number: ${{ github.event.issue.number }} | ||
| token: ${{ secrets.GITHUB_TOKEN }} | ||
| issue-number: ${{ github.event.pull_request.number }} | ||
| body: | | ||
| Hi @${{ github.event.issue.user.login }}! | ||
| Thanks for opening this issue, would you like to work on this? | ||
| 🎉 **Welcome @${{ github.actor }}!** | ||
| Thank you for your pull request! Our team will review it soon. 🔍 | ||
| - Please ensure your PR follows the contribution guidelines. ✅ | ||
| - All automated tests should pass before merging. 🔄 | ||
| - If this PR fixes an issue, link it in the description. 🔗 | ||
|
|
||
| We appreciate your contribution! 🚀 | ||
| reactions: | | ||
| heart | ||
|
|
||
| - name: Send greeting message for PRs | ||
| if: github.event_name == 'pull_request' | ||
| uses: actions-ecosystem/[email protected] | ||
| continue-on-error: true | ||
| - name: Handle Closed PRs | ||
| if: github.event_name == 'pull_request_target' && github.event.action == 'closed' | ||
| uses: peter-evans/create-or-update-comment@v4 | ||
| with: | ||
| github_token: ${{ secrets.GITHUB_TOKEN }} | ||
| repo: ${{ github.repository }} | ||
| number: ${{ github.event.pull_request.number }} | ||
| token: ${{ secrets.GITHUB_TOKEN }} | ||
| issue-number: ${{ github.event.pull_request.number }} | ||
| body: | | ||
| Hi @${{ github.event.pull_request.user.login }}! 🎉 | ||
| Thanks for opening this pull request. We appreciate your contribution. | ||
| ✅ **PR Closed - Thank You, @${{ github.event.pull_request.user.login }}!** | ||
| - If this PR was merged: **Congratulations! Your contribution is now part of the project.** 🚀 | ||
| - If this PR was closed without merging: **Don’t worry! You can always improve it and submit again.** 💪 | ||
|
|
||
| - name: Log completion | ||
| run: | | ||
| echo "Greeting workflow completed successfully" | ||
| echo "Event type: ${{ github.event_name }}" | ||
| We appreciate your effort and look forward to more contributions from you! 🤝 | ||
| reactions: | | ||
| heart | ||
| hooray | ||
| rocket | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,44 @@ | ||
| ## Github Actions CI workflow to deploy to run automated software tests | ||
| name: RUN_SOFTWARE_TESTS | ||
|
|
||
| on: | ||
| # Run this workflow when any pull requests are opened | ||
| workflow_dispatch: | ||
| pull_request: | ||
| types: [opened, synchronize, reopened] | ||
|
|
||
| permissions: | ||
| pull-requests: write | ||
| contents: read | ||
|
|
||
| jobs: | ||
| run_tests: | ||
| name: Run Software Tests | ||
| runs-on: ubuntu-latest | ||
| env: | ||
| # Setup env variables that will be used throughout the workflow | ||
| JAVA_VERSION: 21.0.6 | ||
| FLUTTER_VERSION: 3.35.2 | ||
| steps: | ||
| # Checkout repository codebase | ||
| - name: Checkout the code | ||
| uses: actions/checkout@v4 | ||
|
|
||
| # Setup Java in the VM | ||
| - name: Setup Java to compile the Android project | ||
| uses: actions/setup-java@v4 | ||
| with: | ||
| distribution: "zulu" | ||
| java-version: ${{ env.JAVA_VERSION }} | ||
|
|
||
| # Setup Flutter in the VM | ||
| - name: Setup Flutter | ||
| uses: subosito/flutter-action@v2 | ||
| with: | ||
| flutter-version: ${{ env.FLUTTER_VERSION }} | ||
|
|
||
| - name: 📦 Install dependencies | ||
| run: flutter pub get | ||
|
|
||
| - name: 📉 Run all app tests | ||
| run: flutter test |
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.
Second textarea missing id
Same schema issue here; add an
id.📝 Committable suggestion
🤖 Prompt for AI Agents