Skip to content

Commit 8c2c222

Browse files
brian-chebonABC2015beeme1mrubabar123openfeaturebot
authored
chore: merge Development main (#56)
## This PR - update code coverage workflow - update folder path for workflow - remove unncessary yaml files - update test code coverage - update ignore and coverage workflow - remove unnecessary workflows --------- Signed-off-by: Jeremy Andrews <[email protected]> Signed-off-by: Michael Beemer <[email protected]> Signed-off-by: Usman <[email protected]> Signed-off-by: ubabar123 <[email protected]> Signed-off-by: Brian Chebon <[email protected]> Signed-off-by: OpenFeature Bot <[email protected]> Co-authored-by: Jeremy Andrews <[email protected]> Co-authored-by: Michael Beemer <[email protected]> Co-authored-by: ubabar123 <[email protected]> Co-authored-by: ABC2015 <[email protected]> Co-authored-by: OpenFeature Bot <[email protected]>
1 parent e8f1e9e commit 8c2c222

8 files changed

+506
-277
lines changed

.github/workflows/contributor-workflow.yml

+3-6
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@ jobs:
2828
- name: Install Dependencies
2929
run: |
3030
echo "Installing dependencies for core SDK..."
31-
cd dart-server-sdk-openfeature
3231
if ! dart pub get; then
3332
echo "❌ Failed to install dependencies for the core SDK. Exiting..."
3433
exit 1
@@ -37,13 +36,11 @@ jobs:
3736
- name: Check for Outdated Dependencies
3837
run: |
3938
echo "Checking for outdated dependencies..."
40-
cd dart-server-sdk-openfeature
4139
dart pub outdated || true
4240
4341
- name: Run Static Analysis
4442
run: |
4543
echo "Running static analysis for core SDK..."
46-
cd dart-server-sdk-openfeature
4744
if ! dart analyze lib/ > analysis_report.txt 2> error_log.txt; then
4845
echo "❌ Static analysis failed with errors. See details below:"
4946
cat error_log.txt
@@ -55,7 +52,7 @@ jobs:
5552
- name: Categorize and Count Analysis Results
5653
run: |
5754
echo "Categorizing analysis results..."
58-
cd dart-server-sdk-openfeature
55+
5956
grep -i "info" analysis_report.txt > info_report.txt || echo "No informational messages found."
6057
grep -i "warning" analysis_report.txt > warning_report.txt || echo "No warnings found."
6158
grep -i "error" analysis_report.txt > error_report.txt || echo "No errors found."
@@ -89,7 +86,7 @@ jobs:
8986
- name: Identify Specific Commits for Issues
9087
run: |
9188
echo "Identifying specific commits for issues in the core SDK..."
92-
cd dart-server-sdk-openfeature
89+
9390
while IFS= read -r line; do
9491
file=$(echo $line | awk -F ':' '{print $1}')
9592
if [[ -n "$file" ]]; then
@@ -104,7 +101,7 @@ jobs:
104101
105102
- name: Upload Analysis Artifacts
106103
if: always()
107-
uses: actions/upload-artifact@v3
104+
uses: actions/upload-artifact@v4
108105
with:
109106
name: core-sdk-analysis
110107
path: |

.github/workflows/coverage-workflow.yml

+48-3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
# This workflow valides that branch and commit names are standardized for better workflow.
2-
31
name: Coverage Workflow
42

53
on:
@@ -13,16 +11,63 @@ on:
1311
type: string
1412

1513
jobs:
14+
test-and-upload:
15+
runs-on: ubuntu-latest
16+
outputs:
17+
coverage-path: ${{ steps.upload_artifact.outputs.artifact-path }}
18+
steps:
19+
- name: Checkout code
20+
uses: actions/checkout@v3
21+
22+
- name: Set up Dart
23+
uses: dart-lang/setup-dart@v1
24+
with:
25+
sdk: stable
26+
27+
- name: Install dependencies
28+
run: dart pub get
29+
30+
- name: Run tests with coverage
31+
run: dart test --coverage=coverage
32+
33+
- name: Generate LCOV report
34+
run: |
35+
dart pub global activate coverage
36+
dart run coverage:format_coverage \
37+
--report-on=lib \
38+
--lcov \
39+
-i coverage \
40+
--out=coverage/lcov.info
41+
42+
- name: List coverage files
43+
run: ls -al coverage
44+
45+
- name: Upload coverage artifact
46+
id: upload_artifact
47+
uses: actions/upload-artifact@v4
48+
with:
49+
name: coverage-report
50+
path: coverage/lcov.info
51+
1652
upload-coverage:
1753
runs-on: ubuntu-latest
1854
needs: test-and-upload
1955
steps:
2056
- name: Checkout code
2157
uses: actions/checkout@v3
2258

59+
- name: Download coverage artifact
60+
uses: actions/download-artifact@v4
61+
with:
62+
name: coverage-report
63+
path: ./coverage
64+
65+
- name: List coverage files after download
66+
run: ls -al coverage
67+
2368
- name: Upload coverage reports to Codecov
2469
uses: codecov/codecov-action@v5
2570
with:
26-
token: ${{ secrets.CODECOV_UPLOAD_TOKEN }}
71+
token: ${{ secrets.CODECOV_TOKEN }}
2772
slug: open-feature/dart-server-sdk
2873
files: ./coverage/lcov.info

.github/workflows/main-workflow.yml

-76
This file was deleted.

.github/workflows/parent-pipeline.yml

+15-49
Original file line numberDiff line numberDiff line change
@@ -37,54 +37,20 @@ jobs:
3737
branch_name: ${{ needs.determine-workflow.outputs.branch_name }}
3838
event_name: ${{ needs.determine-workflow.outputs.event_name }}
3939

40-
# trigger-contributor-workflow:
41-
# name: Trigger Contributor Workflow
42-
# needs: trigger-validation
43-
# if: |
44-
# github.event_name == 'push' &&
45-
# needs.determine-workflow.outputs.branch_name != 'main' &&
46-
# needs.determine-workflow.outputs.branch_name != 'qa' &&
47-
# needs.determine-workflow.outputs.branch_name != 'beta'
48-
# uses: ./.github/workflows/contributor-workflow.yml
49-
# with:
50-
# branch_name: ${{ needs.determine-workflow.outputs.branch_name }}
51-
# event_name: ${{ needs.determine-workflow.outputs.event_name }}
5240

53-
# trigger-qa-workflow:
54-
# name: Trigger QA Workflow
55-
# needs: trigger-validation
56-
# if: |
57-
# github.event_name == 'pull_request' &&
58-
# github.base_ref == 'qa'
59-
# uses: ./.github/workflows/qa-workflow.yml
60-
# with:
61-
# branch_name: ${{ needs.determine-workflow.outputs.branch_name }}
62-
# event_name: ${{ needs.determine-workflow.outputs.event_name }}
63-
# trigger-development-workflow:
64-
# name: Trigger Development Workflow
65-
# needs: trigger-validation
66-
# if: |
67-
# github.event_name == 'pull_request' &&
68-
# github.base_ref == 'development'
69-
# uses: ./.github/workflows/development-workflow.yml
70-
# trigger-beta-workflow:
71-
# name: Trigger Beta Workflow
72-
# needs: trigger-validation
73-
# if: |
74-
# github.event_name == 'pull_request' &&
75-
# github.base_ref == 'beta'
76-
# uses: ./.github/workflows/beta-workflow.yml
77-
# with:
78-
# branch_name: ${{ needs.determine-workflow.outputs.branch_name }}
79-
# event_name: ${{ needs.determine-workflow.outputs.event_name }}
41+
trigger-coverage:
42+
name: Trigger Coverage Workflow
43+
needs: determine-workflow
44+
uses: ./.github/workflows/coverage-workflow.yml
45+
with:
46+
branch_name: ${{ needs.determine-workflow.outputs.branch_name }}
47+
event_name: ${{ needs.determine-workflow.outputs.event_name }}
8048

81-
# trigger-main-workflow:
82-
# name: Trigger Main Workflow
83-
# needs: trigger-validation
84-
# if: |
85-
# github.event_name == 'pull_request' &&
86-
# github.base_ref == 'main'
87-
# uses: ./.github/workflows/main-workflow.yml
88-
# with:
89-
# branch_name: ${{ needs.determine-workflow.outputs.branch_name }}
90-
# event_name: ${{ needs.determine-workflow.outputs.event_name }}
49+
50+
contributor-workflow:
51+
name: Trigger Contributor Workflow
52+
needs: determine-workflow
53+
uses: ./.github/workflows/contributor-workflow.yml
54+
with:
55+
branch_name: ${{ needs.determine-workflow.outputs.branch_name }}
56+
event_name: ${{ needs.determine-workflow.outputs.event_name }}

.github/workflows/qa-workflow.yml

-118
This file was deleted.

.gitignore

+9-1
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,10 @@
1-
.dart_tool/
1+
# Global Repo Rules
2+
3+
# Ignore all IDE settings
4+
**/.idea/
5+
**/.vscode/
6+
7+
# Ignore Dart local build tools
8+
**/.dart_tool/
29
**/pubspec.lock
10+
coverage/

0 commit comments

Comments
 (0)