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 semaphore.yml #21

Closed
wants to merge 39 commits into from
Closed

Add semaphore.yml #21

wants to merge 39 commits into from

Conversation

trench8891
Copy link
Owner

@trench8891 trench8891 commented Jun 4, 2019

Resolves #20

  • include semaphore file
  • syntax checks
    • have semaphore run the script
    • include usage message
    • include flags
    • check ALL syntax requirements from kOS docs
  • style checks
    • have semaphore run the script
    • include usage message
    • include flags
    • check line lengths
    • check lazy global off
    • check explicit function scopes
    • check instruction and keyword capitalization
    • check files end with newline
  • workflow checks
    • have semaphore run the script
    • include usage message
    • include flags
    • check for remaining TODOs
    • check for version bump
    • check for CHANGELOG update

@trench8891 trench8891 closed this Dec 11, 2022
@trench8891 trench8891 deleted the semaphore-setup branch December 11, 2022 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement kerboscript linter
1 participant