Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Add a couple of file types to the text section#17670

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
pwilkin wants to merge1 commit intoggml-org:master
base:master
Choose a base branch
Loading
frompwilkin:add-file-types

Conversation

@pwilkin
Copy link
Collaborator

Fixes#17556

I believe we should at least support the source files in the llama.cpp codebase ;)

Copy link
Collaborator

@allozaurallozaur left a comment

Choose a reason for hiding this comment

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

@pwilkin looking good in general.

Please runnpm run format for linting CI to pass + please update the static build :)

@pwilkin
Copy link
CollaboratorAuthor

@pwilkin looking good in general.

Please runnpm run format for linting CI to pass + please update the static build :)

Just when I was getting used to runningeditorconfig-checker on each run, another linter... 😆

@allozaur
Copy link
Collaborator

Just when I was getting used to runningeditorconfig-checker on each run, another linter... 😆

Did you runnpm run dev locally? It should install git hooks where we have a pre-commit hook checking the linting, types etc.

@pwilkin
Copy link
CollaboratorAuthor

Did you runnpm run dev locally? It should install git hooks where we have a pre-commit hook checking the linting, types etc.

Nope, just npm run build :>

@allozaur
Copy link
Collaborator

Did you runnpm run dev locally? It should install git hooks where we have a pre-commit hook checking the linting, types etc.

Nope, just npm run build :>

Busted! 😝 lemme know when you have formatted the code and maybe add a short test video to the PR description? 😊

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@allozaurallozaurallozaur requested changes

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Misc. bug: Regression on file uploads

2 participants

@pwilkin@allozaur
[8]ページ先頭
©2009-2025 Movatter.jp