jmakhack / myanimelist-cli

Minimalistic command line interface for fetching user anime data from MyAnimeList.
https://aur.archlinux.org/packages/mya-git
MIT License
11 stars 15 forks source link

Security-Fix-70/69 #77

Closed The-Debarghya closed 1 year ago

The-Debarghya commented 1 year ago

Associated Issue

Closes #70 #69

Implemented Solution

welcome[bot] commented 1 year ago

Thanks for opening this pull request for myanimelist-cli! Please wait shortly for someone to review. https://media1.tenor.com/images/de54dcf0d5723e5c190b36aed008917f/tenor.gif

ghost commented 1 year ago
👇 Click on the image for a new way to code review - Make big changes easier — review code in small groups of related files - Know where to start — see the whole change at a glance - Take a code tour — explore the change with an interactive tour - Make comments and review — all fully sync’ed with github [Try it now!](https://app.codesee.io/r/reviews?pr=77&src=https%3A%2F%2Fgithub.com%2Fjmakhack%2Fmyanimelist-cli)

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

The-Debarghya commented 1 year ago

I guess I forgot that I had to add libbsd as a dependency, I have added that to the workflow and the makefile.

jmakhack commented 1 year ago

@The-Debarghya Can you take a look at the following build error?

Run make all
gcc -c -g -Wall src/mya.c -o obj/mya.o
src/mya.c: In function ‘generate_endpoint’:
src/mya.c:253:3: error: a label can only be part of a statement and a declaration is not a statement
  253 |   const char* s = "completed";
      |   ^~~~~
make: *** [Makefile:41: obj/mya.o] Error 1
Error: Process completed with exit code 2.

Can you also leave a comment in #69 so I can assign that issue to you too :)

The-Debarghya commented 1 year ago

Can it be merged now?

jmakhack commented 1 year ago

@all-contributors please add @The-Debarghya for code

allcontributors[bot] commented 1 year ago

@jmakhack

I've put up a pull request to add @The-Debarghya! :tada:

welcome[bot] commented 1 year ago

Congrats on merging your first pull request to myanimelist-cli! Please give our project a star if you like what we are building here. https://media1.tenor.com/images/7eaebbe5e4d63fc9b5f029d0bab363e3/tenor.gif