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

Add the GitHub Super Linter #21

Merged
merged 1 commit into from
May 23, 2021
Merged

Conversation

jbampton
Copy link
Member

@jbampton jbampton commented May 3, 2021

Add pre-commit framework.

Add GitHub actions to check spelling, trailing whitespace and merge conflicts.

Add contributing guide.

Remove trailing whitespace.

Fix markdownlint errors.

@jbampton
Copy link
Member Author

Sorry @BaseMax I will fix this I just noticed mate

@jbampton jbampton self-assigned this May 23, 2021
@jbampton jbampton added the Enhancement New feature or request label May 23, 2021
@BaseMax
Copy link
Member

BaseMax commented May 23, 2021

Thanks, there are some Conflicting.

@jbampton
Copy link
Member Author

Is this for project Alpha ?

Screen Shot 2021-05-23 at 12 28 35 pm

@BaseMax
Copy link
Member

BaseMax commented May 23, 2021

Alpha project? I don't know anything about this.

@BaseMax
Copy link
Member

BaseMax commented May 23, 2021

What's required in this repo. we need a perfect C lint. Auto beautify and suggest c bug.

@jbampton
Copy link
Member Author

Great to know about that. Please assign me to that C lint task as an issue.

I think I can do something like that as I know a lot about linters.

Just need a little time to research C a bit more.

@BaseMax BaseMax marked this pull request as draft May 23, 2021 02:33
@BaseMax BaseMax mentioned this pull request May 23, 2021
Add pre-commit framework.

Add GitHub actions to check spelling, trailing whitespace and merge conflicts.

Add contributing guide.

Remove trailing whitespace.

Fix markdownlint errors.
@jbampton
Copy link
Member Author

It seems GitHub actions don't run when first added to the repo.

So we need to merge this first and then on the next pull request the build and tests will run

@BaseMax BaseMax marked this pull request as ready for review May 23, 2021 03:26
@BaseMax BaseMax merged commit 176d71e into One-Language:master May 23, 2021
@jbampton
Copy link
Member Author

I set this up to only run on pull requests that is all the tests

@jbampton jbampton deleted the add-linters branch May 23, 2021 03:28
@BaseMax
Copy link
Member

BaseMax commented May 23, 2021

CONTRIBUTING.md file need to change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants