Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI Fix: Removes multiple newlines at the end of files that breaks editorconfig #8258

Merged
merged 1 commit into from
Jul 2, 2024

Conversation

HanClinto
Copy link
Collaborator

No idea why it's acting up now all of a sudden, but editorconfig is complaining about multiple newlines at the end of files. Went through and removed the extra newlines (but still left one in each place).

Hopefully this fixes CI again.

@github-actions github-actions bot added script Script related testing Everything test related Nvidia GPU Issues specific to Nvidia GPUs Vulkan Issues specific to the Vulkan backend examples python python script changes devops improvements to build systems and github actions server ggml changes relating to the ggml tensor library for machine learning Apple Metal https://en.wikipedia.org/wiki/Metal_(API) labels Jul 2, 2024
@HanClinto HanClinto merged commit 07a3fc0 into ggerganov:master Jul 2, 2024
52 checks passed
@HanClinto HanClinto deleted the remove-trailing-newlines branch July 2, 2024 16:25
Nexesenex pushed a commit to Nexesenex/croco.cpp that referenced this pull request Jul 2, 2024
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Apple Metal https://en.wikipedia.org/wiki/Metal_(API) devops improvements to build systems and github actions examples ggml changes relating to the ggml tensor library for machine learning Nvidia GPU Issues specific to Nvidia GPUs python python script changes script Script related server testing Everything test related Vulkan Issues specific to the Vulkan backend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants