Skip to content

FEA-1676: Prep for open source release #31

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.

Sign up for GitHub

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

Merged
merged 3 commits into from
Apr 7, 2023
Merged

FEA-1676: Prep for open source release #31

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
d0cae64
prep for open source release
matthewnitschke-wk Apr 7, 2023
748eb6e
updated to use ddev format
matthewnitschke-wk Apr 7, 2023
4a08ede
fixed broken snapshots
matthewnitschke-wk Apr 7, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 2 additions & 12 deletions .github/workflows/checks.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,28 +8,18 @@ jobs:
runs-on: workiva-runner-dev
steps:
- uses: actions/checkout@v3

- uses: dart-lang/setup-dart@v1
with:
sdk: stable

- run: dart pub get
- uses: Workiva/gha-dart/dart-format@master
- run: dart run dart_dev format

dependency-validator:
runs-on: workiva-runner-dev
steps:
- uses: actions/checkout@v3

- uses: dart-lang/setup-dart@v1
with:
sdk: stable

- run: dart pub get

# TODO: convert to Workiva/gha-dart/dart-dependency-validator@master once
# https://github.com/Workiva/gha-dart/pull/4 merges
- run: dart run dependency_validator
shell: bash


- run: dart run dependency_validator
48 changes: 0 additions & 48 deletions .github/workflows/scip.yaml
View file
Open in desktop

This file was deleted.

33 changes: 12 additions & 21 deletions .github/workflows/tests.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,7 @@ on:

jobs:
snapshots:
runs-on: workiva-runner-dev
name: Snapshots

runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v3
- name: Install scip cli
Expand All @@ -21,17 +19,14 @@ jobs:

- uses: dart-lang/setup-dart@v1
with:
# use a fixed version number so changes to 'stable', dont break snapshot diffs (language version is used in the scip symbol)
sdk: 2.18.7

# run `pub get` for the scip-dart package
- name: Pub get cli
uses: Workiva/gha-dart/pub-get@master
- name: pub get scip-dart package
run: dart pub get

# run `pub get` for `snapshots`
- name: Pub get snapshots dir
uses: Workiva/gha-dart/pub-get@master
with:
package-path: ./snapshots/input/basic-project
- name: pub get snapshots directory
run: dart pub get
working-directory: ./snapshots/input/basic-project

- name: Snapshots Diff Check
run: |
Expand All @@ -50,29 +45,25 @@ jobs:
fi

consumer:
runs-on: workiva-runner-dev
name: Consumer
runs-on: ubuntu-latest

strategy:
matrix:
repo: ["Workiva/wdesk_sdk", "Workiva/syncdeps_dart"]
repo: ["Workiva/over_react", "rrousselGit/provider", "dart-lang/args"]

steps:
# Setup scip-dart
- uses: actions/checkout@v3
- uses: dart-lang/setup-dart@v1
with:
sdk: 2.18.7
- uses: Workiva/gha-dart/pub-get@master

# Setup wdesk_sdk
# Setup repo to run on
- uses: actions/checkout@v3
with:
repo: ${{ matrix.repo }}
path: ${{ matrix.repo }}
- uses: Workiva/gha-dart/pub-get@master
with:
package-path: ${{ matrix.repo }}
- run: dart pub get
working-directory: ${{ matrix.repo }}

- run: |
dart run scip_dart ./${{ matrix.repo }} || exit 1