Skip to content

Fix "Cannot read property 'match' of undefined" - #232

Merged
michaelgira23 merged 3 commits into
masterfrom
match-undefined-bug
Sep 19, 2023
Merged

Fix "Cannot read property 'match' of undefined"#232
michaelgira23 merged 3 commits into
masterfrom
match-undefined-bug

Conversation

@michaelgira23

Copy link
Copy Markdown
Member

No description provided.

@nickbclifford

Copy link
Copy Markdown
Member

what's up with the tests? looks like something from ttypescript, maybe pull in the typia change from #230

on that note, this seems to indicate that our types are wrong if that function is ever receiving undefined

@michaelgira23

Copy link
Copy Markdown
Member Author

Tests have been broken for a while because of node version. Seems like GitHub Actions has deprecated Node v12, so our actions (tests, lint) now run in v16 and everything breaks. Running locally with v12 does get the codebase to compile though ;)

I have been trying to create a Dockerfile and update all our dependencies but haven't finished yet
https://github.com/MyMICDS/MyMICDS-v2/tree/docker-setup

@michaelgira23

Copy link
Copy Markdown
Member Author

Screw it

@michaelgira23
michaelgira23 merged commit 3ec75a7 into master Sep 19, 2023
@michaelgira23
michaelgira23 deleted the match-undefined-bug branch September 19, 2023 06:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants