feat: github action for PR

CapgeminiInventUK · Feb 4, 2024 · 86af5a4 · 86af5a4
1 parent 568fe95
commit 86af5a4
Show file tree

Hide file tree

Showing 5 changed files with 25 additions and 11 deletions.
diff --git a/.github/workflows/pull-request.yml b/.github/workflows/pull-request.yml
@@ -1,4 +1,4 @@
-name: Node.js CI
+name: Pull Request
 
 on: [pull_request]
 
@@ -9,9 +9,9 @@ jobs:
       matrix:
         project: [ui, server]
     steps:
-      - uses: actions/checkout@v2
-      - name: Use Node.js
-        uses: actions/setup-node@v2
+      - uses: actions/checkout@v4
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
         with:
           node-version: '20'
       - name: Install dependencies

diff --git a/server/.eslintrc.json b/server/.eslintrc.json
@@ -11,6 +11,10 @@
     "deprecation",
     "unicorn"
   ],
+  "ignorePatterns": [
+    "node_modules/",
+    "dist/"
+  ],
   "rules": {
     "unicorn/filename-case": "error",
     "indent": [

diff --git a/server/package.json b/server/package.json
@@ -1,7 +1,7 @@
 {
-  "name": "langtrace-ingest",
+  "name": "langtrace-server",
   "version": "1.0.0",
-  "main": "src/ingestor_server.ts",
+  "main": "src/ingestor-server.ts",
   "scripts": {
     "start:langtrace": "ts-node src/langtrace-server.ts",
     "start:ingestor": "ts-node src/ingest-server.ts",

diff --git a/ui/.eslintrc.json b/ui/.eslintrc.json
@@ -3,6 +3,11 @@
   "plugins": [
     "unicorn"
   ],
+  "ignorePatterns": [
+    "node_modules/",
+    ".next/",
+    "dist/"
+  ],
   "rules": {
       "unicorn/filename-case": [
         "error",

diff --git a/ui/src/pages/api/auth/[...nextauth].ts b/ui/src/pages/api/auth/[...nextauth].ts
@@ -1,14 +1,19 @@
-import NextAuth, { NextAuthOptions } from 'next-auth';
+import NextAuth, { NextAuthOptions, Profile } from 'next-auth';
 import GithubProvider from 'next-auth/providers/github';
 import { NextApiRequest, NextApiResponse } from 'next';
 
 const enableAuth = process.env.NEXTAUTH_ENABLE === 'true';
 
+interface GitHubProfile extends Profile {
+  login: string;
+
+}
+
 const options = {
   providers: [
     GithubProvider({
-      clientId: process.env.NEXTAUTH_GITHUB_ID,
-      clientSecret: process.env.NEXTAUTH_GITHUB_SECRET,
+      clientId: process.env.NEXTAUTH_GITHUB_ID!,
+      clientSecret: process.env.NEXTAUTH_GITHUB_SECRET!,
       authorization: {
         params: {
           scope: 'read:user,user:email,read:org',
@@ -20,10 +25,10 @@ const options = {
     async signIn({ account, profile }) {
       if (account?.provider === 'github' &&
         (process.env.NEXTAUTH_GITHUB_ORGANISATION ?? '').trim() !== '') {
-
         const orgName = process.env.NEXTAUTH_GITHUB_ORGANISATION;
         const token = account.access_token;
-        const url = `https://api.github.com/orgs/${orgName}/members/${profile?.login}`;
+        const url =
+          `https://api.github.com/orgs/${orgName}/members/${(profile as GitHubProfile).login}`;
         const response = await fetch(url, {
           headers: {
             Authorization: `token ${token}`,