Skip to content

Commit

Permalink
SYS-192 add github ci
Browse files Browse the repository at this point in the history
update permissions to allow for checkout of private repo in reusable workflow
fixing issue comment Pr review stuff
add codeowners and sync ci.yml

fix lint script, updated format check and fix for naming consistency
  • Loading branch information
TieDyedSheep authored and afostr committed Jul 24, 2024
1 parent e28e9fa commit b075f20
Show file tree
Hide file tree
Showing 3 changed files with 67 additions and 13 deletions.
61 changes: 51 additions & 10 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
name: Node CI Workflow
# The parameters are defaulted at the org level but can be overridden on the repository.
#
# The parameters are defaulted at the org level but can be overridden on the repository.
# See the github-automation repo for more documentation
#
on:
push:
branches:
Expand All @@ -9,6 +12,16 @@ on:
branches:
- dev
- main
issue_comment:
inputs:
workflowBranch:
description: 'Branch of the reusable workflow. Defaults to main, select dev for testing only.'
required: true
default: 'main'
type: choice
options:
- dev
- main
workflow_dispatch:
inputs:
workflowBranch:
Expand All @@ -19,18 +32,46 @@ on:
options:
- dev
- main

permissions:
issues: write
pull-requests: write

jobs:
ci-dev:
echo-inputs:
name: Repo Workflow Debugging
runs-on: ubuntu-latest
steps:
- name: Check Repo Vars
run: |
echo "*** Start - Check inputs in repo workflow ***"
echo "Node Version: ${{ vars.NODE_VERSION }}"
echo "Lint Required: ${{ vars.IS_LINT_REQUIRED }}"
echo "Format Check Required: ${{ vars.IS_FORMAT_CHECK_REQUIRED }}"
echo "Apply Patches Required: ${{ vars.IS_APPLY_PATCHES_REQUIRED }}"
echo "Unit Tests Required: ${{ vars.IS_UNIT_TESTS_REQUIRED }}"
echo "*** End - Check inputs in repo workflow ***"
ci-test-only:
if: ${{ github.event.inputs.workflowBranch == 'dev' }}
uses: shardeum/github-automation/.github/workflows/node-ci-shared.yml@dev
uses: shardeum/github-automation/.github/workflows/reusable-node-ci.yml@dev
permissions:
issues: write
pull-requests: write
contents: write
with:
node-version: ${{ vars.NODE_VERSION }}
lint-required: ${{ vars.IS_LINT_REQUIRED == 'true' }}
format-check-required: ${{ vars.IS_FORMAT_CHECK_REQUIRED == 'true' }}
apply-patches-required: ${{ vars.IS_APPLY_PATCHES_REQUIRED == 'true' }}
unit-tests-required: ${{ vars.IS_UNIT_TESTS_REQUIRED == 'true' }}
secrets: inherit

ci-main:
ci:
if: ${{ github.event.inputs.workflowBranch == 'main' || !github.event.inputs.workflowBranch }}
uses: shardeum/github-automation/.github/workflows/node-ci-shared.yml@main
uses: shardeum/github-automation/.github/workflows/reusable-node-ci.yml@main
permissions:
issues: write
pull-requests: write
contents: write
with:
node-version: ${{ vars.NODE_VERSION }}
lint-required: ${{ vars.IS_LINT_REQUIRED == 'true' }}
format-check-required: ${{ vars.IS_FORMAT_CHECK_REQUIRED == 'true' }}
apply-patches-required: ${{ vars.IS_APPLY_PATCHES_REQUIRED == 'true' }}
unit-tests-required: ${{ vars.IS_UNIT_TESTS_REQUIRED == 'true' }}
secrets: inherit
13 changes: 13 additions & 0 deletions CODEOWNERS
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
# CODEOWNERS file
# To add additional teams to any approval, include them on the same line separated by spaces
# It is best practice to assign a team as a code owner and not an invidual.
# Please submit requests for new teams to Systems and Automation

# Global approval (all files)
# * @shardeum/team-name

# Directory-level approval
/.github/ @shardeum/systems-and-automation

# Specific file rules
# README.md @shardeum/team-name
6 changes: 3 additions & 3 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,9 @@
"compile": "tsc -p .",
"prepare": "npm run compile",
"pretest": "npm run compile",
"lint": "eslint './public/**/*.ts'",
"lint-windows": "eslint ./public/**/*.ts",
"format-check": "prettier --check './public/**/*.ts'"
"lint": "eslint \"./public/**/*.ts\"",
"format-check": "prettier --check './public/**/*.ts'",
"format-fix": "prettier --write './public/**/*.ts'"
},
"repository": {
"type": "git",
Expand Down

0 comments on commit b075f20

Please sign in to comment.