Skip to content

Conversation

@satyamvatsal
Copy link

now font can be installed from the util directory.

Copy link

@antonkesy antonkesy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@satyamvatsal
Copy link
Author

I am new to this repo and GitHub
Can you use full form of LGTM

@DaftOdyssey
Copy link

I am new to this repo and GitHub Can you use full form of LGTM

Sure, LGTM simply means "Looks Good To Me"

Copy link

@barnett-yuxiang barnett-yuxiang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it not good to use the project root path?

@satyamvatsal
Copy link
Author

But most of the people run the script by cd into the repository.
So I think it will be more sensible.

@idan
Copy link
Contributor

idan commented May 9, 2024

Fixed in the batch of commits around 93c4bcd

@idan idan closed this May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants