-
Notifications
You must be signed in to change notification settings - Fork 26
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #99 from surfstudio/SNP-903_fix_and_update_CI
chore(SNP-903): Add CI, add new templates, update FVM
- Loading branch information
Showing
20 changed files
with
378 additions
and
79 deletions.
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
{ | ||
"flutter": "3.19.0" | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
blank_issues_enabled: false | ||
contact_links: | ||
- name: SurfGear telegram chat | ||
url: https://t.me/SurfGear | ||
about: Please ask and answer questions here. |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
--- | ||
name: Documentation update | ||
about: Fixing a problem or improve in an existing docs page. | ||
labels: documentation | ||
--- | ||
<!-- | ||
Thank you for contributing to our project! | ||
This template is only for documentation update reports, otherwise please use another template. | ||
--> | ||
## Specify package name | ||
<!--Write the name of the package where you have problems--> | ||
|
||
## Describe what scenario you think is uncovered by the existing examples / articles | ||
<!--A clear and concise description of the problem that you want explained.--> | ||
|
||
## Describe why existing examples / articles do not cover this case | ||
<!--Explain which examples / articles you have seen before making this request, and why they did not help you with your problem.--> | ||
|
||
## Additional context | ||
<!--Add any other context or screenshots about the documentation request here.--> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
--- | ||
name: Feature request | ||
about: I'd like SurfGear team to do something new. | ||
labels: enhancement | ||
--- | ||
<!-- | ||
Thank you for contributing to our project! | ||
This template is only for feature requests, otherwise please use another template. | ||
Provide thoughtful commentary *and code samples* on what this feature means for your product. What will it allow you to do that you can't do today? How will it make current workarounds straightforward? What potential bugs and edge cases does it help to avoid? etc. Please keep it product-centric. | ||
--> | ||
## Specify package name | ||
<!--Write the name of the package where you have problems--> | ||
|
||
## What is the new or updated feature that you are suggesting? | ||
<!--A clear and concise description of what you want to happen.--> | ||
|
||
## Why should this feature be included? | ||
<!--Is your feature request related to a problem? Please describe a clear and concise description of what the problem is. If you investigate any alternative solutions, please describe.--> | ||
|
||
## Additional context | ||
<!--Please provide any other context or screenshots about the feature request here.--> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,70 +1,6 @@ | ||
## Related tasks | ||
<!-- | ||
For more info https://github.com/surfstudio/flutter-roadmap/blob/main/docs/PULL_REQUEST_TEMPLATE_README.md#related-tasks | ||
--> | ||
Tap on "Preview" ⤴ | ||
|
||
|
||
## Dependence on other Pull Requests (optional) | ||
<!-- | ||
For more info https://github.com/surfstudio/flutter-roadmap/blob/main/docs/PULL_REQUEST_TEMPLATE_README.md#dependence-on-other-pull-requests-optional | ||
--> | ||
|
||
## Background (optional) | ||
<!-- | ||
For more info https://github.com/surfstudio/flutter-roadmap/blob/main/docs/PULL_REQUEST_TEMPLATE_README.md#background-optional | ||
--> | ||
|
||
## Changes | ||
<!-- | ||
For more info https://github.com/surfstudio/flutter-roadmap/blob/main/docs/PULL_REQUEST_TEMPLATE_README.md#changes | ||
--> | ||
|
||
## Troubles (optional) | ||
<!-- | ||
For more info https://github.com/surfstudio/flutter-roadmap/blob/main/docs/PULL_REQUEST_TEMPLATE_README.md#troubles-optional | ||
--> | ||
|
||
## Note (optional) | ||
<!-- | ||
For more info https://github.com/surfstudio/flutter-roadmap/blob/main/docs/PULL_REQUEST_TEMPLATE_README.md#note-optional | ||
--> | ||
|
||
## Checklist for self-check | ||
- [ ] Commits and PRs have been filed according to [the rules on the project](https://github.com/surfstudio/surf-flutter-app-template#workflow-in-a-repository). | ||
- [ ] The author is marked as an assigne and assigned mandatory reviewers. | ||
- [ ] Required labels marked | ||
- [ ] Specified related tasks and/or related PRs. | ||
- [ ] Specified Changes. | ||
- [ ] Attached videos/screenshots demonstrating the fix/feature. | ||
- [ ] All unspecified fields in the PR description deleted. | ||
- [ ] New code covered by tests. | ||
|
||
## Checklist for reviewers | ||
- [ ] CI passed successfully _(with a green check mark)_. | ||
- [ ] PR is atomic, by volume no more than 400 (+-) corrected lines (not including codogen). | ||
|
||
Design: | ||
- [ ] System design corresponds to the agreements on structure and architecture on the project. | ||
- [ ] The code is decomposed into necessary and sufficient components. | ||
|
||
Functionality: | ||
- [ ] The code solves the problem. | ||
- [ ] Any changes to the user interface are reasonable and look good. | ||
|
||
Complexity: | ||
- [ ] The code is clear, easy to read, functions are small, no more than 50 lines. | ||
- [ ] The logic is not overcomplicated, there is no overengineering (no code sections that may be needed in the future, but no one knows about it). | ||
|
||
Tests: | ||
- [ ] Updated or added tests for mandatory components. | ||
- [ ] The tests are correct, helpful, and well designed/developed. | ||
|
||
Naming: | ||
- [ ] The naming of variables, methods, classes and other components is understandable. | ||
|
||
Comments: | ||
- [ ] The comments are understandable and helpful. | ||
|
||
Documentation: | ||
- [ ] All labels are correct | ||
- [ ] Technical documentation updated (after approval, updates last reviewer). | ||
And choose one of the templates: | ||
* [Bugfix PR](?expand=1&template=bug_fix.md) | ||
* [New feature PR](?expand=1&template=new_feature.md) | ||
* [Documentation update PR](?expand=1&template=documentation_update.md) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
<!-- | ||
Thank you for contributing to our project! | ||
Provide a description of your changes below and a general summary in the title. | ||
Please look at the following checklist to ensure that your PR can be accepted quickly: | ||
--> | ||
## Checklist | ||
|
||
- [ ] Have you added an explanation of what your changes do and why you'd like us to include them? | ||
- [ ] Is there an existing issue for this PR? | ||
- _link issue here_ (use keywords like `fix`, `close`, `resolve` etc. if necessary) | ||
- [ ] Have the files been linted and formatted? | ||
- [ ] Have the docs been updated to match the changes in the PR? | ||
- [ ] Have the tests been updated to match the changes in the PR? | ||
- [ ] Attached videos/screenshots demonstrating the fix/feature. | ||
- [ ] Have you run the tests locally to confirm they pass? | ||
|
||
## Changes | ||
|
||
### What is the current behavior, and the steps to reproduce the issue? | ||
|
||
### What is the expected behavior? | ||
|
||
### How does this PR fix the problem? |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
<!-- | ||
Thank you for contributing to our project! | ||
Provide a description of your changes below and a general summary in the title. | ||
Please look at the following checklist to ensure that your PR can be accepted quickly: | ||
--> | ||
## Checklist | ||
|
||
- [ ] Is there an existing issue for this PR? | ||
- _link issue here_ (use keywords like `fix`, `close`, `resolve` etc. if necessary) | ||
- [ ] Have the files been linted and formatted? | ||
|
||
## Changes | ||
|
||
### What docs page needs to be fixed? | ||
|
||
- **Section**: | ||
- **Page**: | ||
|
||
## What is the problem? | ||
|
||
## What changes does this PR make to fix the problem? |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
<!-- | ||
Thank you for contributing to our project! | ||
Provide a description of your changes below and a general summary in the title. | ||
Please look at the following checklist to ensure that your PR can be accepted quickly: | ||
--> | ||
## Checklist | ||
|
||
- [ ] Have you added an explanation of what your changes do and why you'd like us to include them? | ||
- [ ] Is there an existing issue for this PR? | ||
- _link issue here_ (use keywords like `fix`, `close`, `resolve` etc. if necessary) | ||
- [ ] Have the files been linted and formatted? | ||
- [ ] Have the docs been updated to match the changes in the PR? | ||
- [ ] Have the tests been updated to match the changes in the PR? | ||
- [ ] Attached videos/screenshots demonstrating the fix/feature. | ||
- [ ] Have you run the tests locally to confirm they pass? | ||
|
||
## New Features | ||
|
||
### What new capabilities does this PR add? | ||
|
||
### What docs changes are needed to explain this? |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,59 @@ | ||
name: "Analysis" | ||
|
||
on: | ||
workflow_call: | ||
inputs: | ||
flutter-version: | ||
required: false | ||
type: string | ||
|
||
jobs: | ||
check_branch_content: | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: Checkout | ||
uses: actions/checkout@v4 | ||
|
||
- name: Content | ||
id: check_files | ||
uses: andstor/file-existence-action@v3 | ||
with: | ||
files: "LICENSE, README.md, CHANGELOG.md, CONTRIBUTING.md" | ||
|
||
- name: File exists | ||
if: steps.check_files.outputs.files_exists == 'true' | ||
run: echo Content is ok! | ||
|
||
analysis: | ||
runs-on: ubuntu-latest | ||
|
||
steps: | ||
- name: Checkout | ||
uses: actions/checkout@v4 | ||
|
||
- name: Install Flutter with exact version | ||
if: ${{ inputs.flutter-version != ''}} | ||
uses: subosito/flutter-action@v2 | ||
with: | ||
flutter-version: ${{ inputs.flutter-version }} | ||
|
||
- name: Install Flutter from Stable branch | ||
if: ${{ inputs.flutter-version == ''}} | ||
uses: subosito/flutter-action@v2 | ||
with: | ||
channel: "stable" | ||
|
||
- name: Run formatter | ||
run: dart format -l 120 --set-exit-if-changed lib test | ||
|
||
- name: Install dependencies | ||
run: flutter pub get | ||
|
||
- name: Run analyzer | ||
run: flutter analyze --fatal-warnings --fatal-infos . | ||
|
||
- name: Run package analyzer | ||
uses: axel-op/dart-package-analyzer@v3 | ||
with: | ||
githubToken: ${{ secrets.GITHUB_TOKEN }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
name: "Checking issue for correctness of described" | ||
|
||
on: | ||
issues: | ||
types: | ||
- reopened | ||
- opened | ||
|
||
jobs: | ||
check-issue: | ||
if: github.repository == 'surfstudio/surf-flutter-app-template' | ||
uses: surfstudio/flutter-ci-workflows/.github/workflows/check_issues.yaml@main | ||
with: | ||
REPO_PATH: 'https://github.com/surfstudio/surf-flutter-app-template/issues/new/choose' | ||
ISSUE_TITLE: ${{ github.event.issue.title }} | ||
ISSUE_BODY: ${{ github.event.issue.body }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,34 @@ | ||
name: "Main" | ||
|
||
on: | ||
pull_request: | ||
|
||
jobs: | ||
get_fvm_version: | ||
if: github.repository == 'surfstudio/surf-flutter-app-template' | ||
outputs: | ||
flutter_version: ${{ steps.fvm_version.outputs.flutter }} | ||
name: "Get Flutter version from FVM" | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Checkout | ||
uses: actions/checkout@v4 | ||
|
||
- id: fvm_version | ||
uses: zoexx/github-action-json-file-properties@release | ||
with: | ||
file_path: ".fvmrc" | ||
|
||
analysis: | ||
needs: get_fvm_version | ||
uses: ./.github/workflows/analysis.yml | ||
with: | ||
flutter-version: ${{ needs.get_fvm_version.outputs.flutter_version }} | ||
|
||
testing: | ||
needs: [analysis, get_fvm_version] | ||
uses: surfstudio/flutter-ci-workflows/.github/workflows/testing.yml@main | ||
with: | ||
flutter-version: ${{ needs.get_fvm_version.outputs.flutter_version }} | ||
secrets: | ||
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,36 @@ | ||
name: "Create Pyrus task from issue" | ||
|
||
on: | ||
issues: | ||
types: | ||
- reopened | ||
- opened | ||
|
||
jobs: | ||
get_package_name: | ||
if: github.repository == 'surfstudio/surf-flutter-app-template' | ||
name: Get package name | ||
runs-on: ubuntu-latest | ||
outputs: | ||
package_name: ${{ steps.parse_pubspec.outputs.name }} | ||
steps: | ||
- name: Checkout | ||
uses: actions/checkout@v4 | ||
- name: Parsing pubspec.yaml | ||
id: parse_pubspec | ||
uses: actions-tools/yaml-outputs@v2 | ||
with: | ||
file-path: 'pubspec.yaml' | ||
|
||
call_create_pyrus_task_from_issue: | ||
name: Create Pyrus task from issue | ||
needs: get_package_name | ||
uses: surfstudio/flutter-ci-workflows/.github/workflows/new_issue_to_pyrus_task.yml@main | ||
with: | ||
PACKAGE_NAME: ${{ needs.get_package_name.outputs.package_name }} | ||
ISSUE_TITLE: ${{ github.event.issue.title }} | ||
ISSUE_BODY: ${{ github.event.issue.body }} | ||
ISSUE_LINK: ${{ github.event.issue.html_url }} | ||
secrets: | ||
LOGIN: ${{ secrets.PYRUS_BOT_LOGIN }} | ||
SECURITY_KEY: ${{ secrets.PYRUS_BOT_SECRET_KEY }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
# This workflow uses actions that are not certified by GitHub. | ||
# They are provided by a third-party and are governed by | ||
# separate terms of service, privacy policy, and support | ||
# documentation. | ||
|
||
name: Label issues | ||
on: | ||
issues: | ||
types: | ||
- reopened | ||
- opened | ||
jobs: | ||
label_issues: | ||
if: github.repository == 'surfstudio/surf-flutter-app-template' | ||
uses: surfstudio/flutter-ci-workflows/.github/workflows/new_issues_label.yml@main | ||
secrets: | ||
SECRET_ISSUES: ${{ secrets.ACCESS_TOKEN_FOR_SURFGEAR }} |
Oops, something went wrong.