Skip to content
This repository has been archived by the owner on Feb 22, 2024. It is now read-only.

Structure Overhaul #4

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Structure Overhaul #4

wants to merge 14 commits into from

Conversation

NuroDev
Copy link
Owner

@NuroDev NuroDev commented Jun 11, 2022

What's been changed?

  • Overhauled project structure to more modular module structure. Example structure:
    • MODULE_NAME
      • index.ts: Exports module function & types
      • MODULE_NAME.module.ts: Actual module handler / fetching logic
      • MODULE_NAME.test.ts: Basic testing for this specific module
      • MODULE_NAME.types.ts: Types specific to this module & the PUBG API
  • Added /types & /constants exports to package.json
  • Added basic testing used Vitest
  • Added CI GitHub workflow
  • Removed example Vite project
  • Fixed some types (Game, Platform, etc)
  • Added environment.d.ts to add PUBG_API_KEY to process.env
  • Added import path aliases to tsconfig.json
  • Upgraded dependencies

@NuroDev NuroDev self-assigned this Jun 11, 2022
@vercel
Copy link

vercel bot commented Jun 11, 2022 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
pubg-ts ❌ Failed (Inspect) Jun 11, 2022 at 2:14AM (UTC)

@DanBennettUK DanBennettUK marked this pull request as ready for review June 22, 2022 15:36
@DanBennettUK
Copy link
Collaborator

I DID NOT CLICK THAT

@NuroDev
Copy link
Owner Author

NuroDev commented Jun 22, 2022

😂

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@NuroDev NuroDev

Labels
work-in-progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@NuroDev @DanBennettUK