-
Notifications
You must be signed in to change notification settings - Fork 670
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[Monorepo] Fix flyteidl generated code and scripts (#4173)
* Remove update_boilerplate and fix references to flytestdlib Signed-off-by: Eduardo Apolinario <[email protected]> * make -C flyteidl generate Signed-off-by: Eduardo Apolinario <[email protected]> * Add Flyteidl verification tests Signed-off-by: Eduardo Apolinario <[email protected]> * Only run flyteidl checks in flyteidl-checks.yml Signed-off-by: Eduardo Apolinario <[email protected]> * Use flytestdlib pflags binary directly Signed-off-by: Eduardo Apolinario <[email protected]> * Fix tests to work with newer grpc library Signed-off-by: Eduardo Apolinario <[email protected]> * Remove unused gh workflow steps Signed-off-by: Eduardo Apolinario <[email protected]> * A test Signed-off-by: Eduardo Apolinario <[email protected]> * Revert "A test" This reverts commit e46f6cc. Signed-off-by: Eduardo Apolinario <[email protected]> * Build component images if flyteidl changes on pushes to master Signed-off-by: Eduardo Apolinario <[email protected]> --------- Signed-off-by: Eduardo Apolinario <[email protected]> Co-authored-by: Eduardo Apolinario <[email protected]>
- Loading branch information
1 parent
c88f8f5
commit 4ee73f5
Showing
205 changed files
with
1,646 additions
and
1,908 deletions.
There are no files selected for viewing
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,52 @@ | ||
name: Flyteidl Verification Tests | ||
|
||
on: | ||
pull_request: | ||
paths: | ||
- 'flyteidl/**' | ||
push: | ||
branches: | ||
- master | ||
paths: | ||
- 'flyteidl/**' | ||
env: | ||
GO_VERSION: "1.19" | ||
jobs: | ||
unpack-envvars: | ||
runs-on: ubuntu-latest | ||
outputs: | ||
go-version: ${{ steps.step.outputs.go-version }} | ||
steps: | ||
- id: step | ||
run: | | ||
echo "go-version=${{ env.GO_VERSION }}" >> $GITHUB_OUTPUT | ||
lint: | ||
name: Lint | ||
needs: | ||
- unpack-envvars | ||
strategy: | ||
fail-fast: false | ||
uses: ./.github/workflows/lint.yml | ||
with: | ||
component: flyteidl | ||
go-version: ${{ needs.unpack-envvars.outputs.go-version }} | ||
unit-tests: | ||
name: Unit Tests | ||
needs: | ||
- unpack-envvars | ||
uses: ./.github/workflows/unit-tests.yml | ||
with: | ||
component: flyteidl | ||
go-version: ${{ needs.unpack-envvars.outputs.go-version }} | ||
secrets: | ||
FLYTE_BOT_PAT: ${{ secrets.FLYTE_BOT_PAT }} | ||
generate: | ||
name: Check Go Generate | ||
needs: | ||
- unpack-envvars | ||
strategy: | ||
fail-fast: false | ||
uses: ./.github/workflows/go_generate.yml | ||
with: | ||
component: flyteidl | ||
go-version: ${{ needs.unpack-envvars.outputs.go-version }} |
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
Oops, something went wrong.