Skip to content
This repository has been archived by the owner on Jan 17, 2024. It is now read-only.

Add CI #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add CI #3

wants to merge 1 commit into from

Conversation

theHamsta
Copy link
Member

No description provided.

@theHamsta theHamsta closed this Apr 1, 2021
@theHamsta theHamsta reopened this Apr 1, 2021
@theHamsta theHamsta closed this Apr 2, 2021
@theHamsta theHamsta reopened this Apr 2, 2021
@theHamsta theHamsta marked this pull request as draft April 2, 2021 00:07
@theHamsta
Copy link
Member Author

Apparently, CI does not run on templates?

@theHamsta theHamsta closed this Apr 2, 2021
@theHamsta theHamsta reopened this Apr 2, 2021
@theHamsta theHamsta marked this pull request as ready for review April 2, 2021 14:34
@theHamsta
Copy link
Member Author

The CI runs when I create a PR on my fork...

@theHamsta theHamsta requested a review from stsewd July 19, 2021 23:28
@@ -0,0 +1,3 @@
#!/usr/bin/env bash

luacheck `find -name "*.lua"` --codes
Copy link
Member

Choose a reason for hiding this comment

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

You can write this as, and if you need to ignore files, you can set those in the config file as https://github.com/nvim-treesitter/nvim-treesitter/blob/dedd74ebdbb8a921d95e3c670be9684d9f655c34/.luacheckrc#L5-L7, also you can add the --codes option in the config file https://github.com/nvim-treesitter/nvim-treesitter/blob/dedd74ebdbb8a921d95e3c670be9684d9f655c34/.luacheckrc#L3

Suggested change
luacheck `find -name "*.lua"` --codes
luacheck .

@stsewd
Copy link
Member

stsewd commented Jul 30, 2021

also, would be great to have stylua, but not strong opinion there

@theHamsta
Copy link
Member Author

This PR is very old feel free to update it to the current CI of the other repos!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants