Skip to content
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.

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

refactor: Migrate project management to Projen #162

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 8 commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
dfb30e1
WIP: refactor to projen for improved package management
flamingquaks Oct 27, 2024
296b3a2
refactor: shifting project to projen
flamingquaks Oct 28, 2024
b25cad9
chore: re-added commitizen configuration to package.json via projen
flamingquaks Oct 28, 2024
aa13981
fix: previous commits accidentally ignored lib/ files and other impor…
flamingquaks Oct 28, 2024
ba4dd53
ci: fixed build issues with ci
flamingquaks Oct 28, 2024
ea36b5e
ci: fixed issue with build deps
flamingquaks Oct 30, 2024
8bfde12
ci: fixed build-on-every-push GitHub Action
flamingquaks Oct 30, 2024
14ae862
fix(projen): Fixed build issues with front end compilation
flamingquaks Oct 30, 2024
d0af5fb
fix: added missing dev dependencies
flamingquaks Oct 30, 2024
0c4ed2a
Fix code scanning alert no. 518: Inefficient regular expression
flamingquaks Oct 30, 2024
cfa66e7
fix: added missing dev dependencies to .projenrc.ts
flamingquaks Oct 30, 2024
4e485c1
fix: added missing eslint plugin
flamingquaks Oct 30, 2024
3c3a145
fix(ci): eslint GitHub Actions update for projen
flamingquaks Oct 30, 2024
a62847b
chore: upgraded dependencies
flamingquaks Oct 30, 2024
83626b6
ci(gitlab): updated GitLab CI script to use npm run ci:deploy
flamingquaks Oct 30, 2024
bd351e3
ci(gitlab): fixed build issue on GitLab due to updates from projen re…
flamingquaks Oct 30, 2024
e7963d8
docs: updated Deployment guide to be aligned to projen refactor
flamingquaks Oct 30, 2024
3537d0a
fix: updated logging bucket ACL settings in CDK to handle log deliver…
flamingquaks Oct 30, 2024
e0c56a4
chore: fixed a missing semicolon
flamingquaks Oct 30, 2024
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
243 changes: 224 additions & 19 deletions .eslintrc.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

18 changes: 18 additions & 0 deletions .gitattributes

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

91 changes: 27 additions & 64 deletions .github/workflows/build-on-every-push.yml
Original file line number Diff line number Diff line change
@@ -1,76 +1,39 @@
# This workflow is designed to build and synthesize each push to reduce risk of failing builds moving into main
name: Build and Synthesize


on:
push:
branches:
- '**'
workflow_dispatch:
workflow_dispatch:
jobs:

build:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v4

- name: Setup Node.js
uses: actions/setup-node@v4
with:
node-version: 20

- name: Install dependencies
run: npm ci

- name: Build project
run: npm run build

- name: Upload UI Artifact
uses: actions/upload-artifact@v4
with:
name: built-ui
path: lib/user-interface/genai-newsletter-ui/dist/
- name: Upload AppSync Resolver Artifacts
uses: actions/upload-artifact@v4
with:
name: built-resolver-functions
path: lib/api/functions/out

synthesis:
needs: build
runs-on: ubuntu-latest


steps:
- uses: actions/checkout@v4

- name: Setup Node.js
uses: actions/setup-node@v4
with:
node-version: 20

- name: Install dependencies
run: npm ci

- name: Download UI Artifact
uses: actions/download-artifact@v4
with:
name: built-ui
path: lib/user-interface/genai-newsletter-ui/dist/
merge-multiple: true

- name: Download Resolver Function Artifacts
uses: actions/download-artifact@v4
with:
name: built-resolver-functions
path: lib/api/functions/out/
merge-multiple: true

- name: Generate Configuration file
run: echo "${DEPLOY_CONFIG}" > ./bin/config.json
env:
DEPLOY_CONFIG: ${{ vars.DEPLOY_CONFIG }}
- uses: actions/checkout@v4

- name: Setup Node.js
uses: actions/setup-node@v4
with:
node-version: 20
- name: Generate Configuration file
run: echo "${DEPLOY_CONFIG}" > ./bin/config.json
env:
DEPLOY_CONFIG: ${{ vars.DEPLOY_CONFIG }}
- name: Install Dependencies
run: npm ci

- name: Synthesize CDK
run: npx cdk synth

- name: Build project
run: npm run build

- name: Upload UI Artifact
uses: actions/upload-artifact@v4
with:
name: built-ui
path: lib/user-interface/genai-newsletter-ui/dist/
- name: Upload AppSync Resolver Artifacts
uses: actions/upload-artifact@v4
with:
name: built-resolver-functions
path: lib/api/functions/out
Loading
Loading