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 CircleCI config #1

Closed
wants to merge 2 commits into from
Closed

Conversation

jeremyBanks
Copy link
Collaborator

For your consideration, if this would be useful to you.

Adds a CircleCI config (as JSON) which will try to build, run tests, and verify formatting on every push, and will also check for warnings and outdated dependencies on pushes to master (after you enable it at https://circleci.com/add-projects/gh/cronning9).

Adds an initial minimal test: that the program can lex a simple string without panicking.

@cronning9
Copy link
Owner

cronning9 commented Jul 5, 2019

@jeremyBanks cool stuff, thanks for the leg work! The new Circle configuration format is pretty neat honestly :)

I'll probably merge this in a bit and do some proper unit testing. Will probably set up a bash script to ensure that the CLI command input works as expected as well.

@jeremyBanks jeremyBanks closed this Sep 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants