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

Added Travis CI #5

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

joshua-cerniglia
Copy link

Added Travis CI where it will run the file currently in the tests folder. Also modified dev-requirement texts so it specifies the version of black. Finally, added a comment so the ./tests/init.py is not empty. Once again apologies for multiple commits, could you please squash them when merging the branch.

@duo-kfleischman
Copy link
Contributor

Hey Joshua! I'm Kai, an intern at Duo working on this project. Sorry for the slow reply. We are working on making a big merge to master happen for this Thursday. Once that change is made you should review your changes, modify them according to the new changes we will have merged, and then we will review what you have. Just so you know, support for Gitlab CI will be added in our Thursday change.

@AaronAtDuo
Copy link

Since Travis CI wants money now, we probably don't want to pull this in.

coltwill pushed a commit that referenced this pull request Jan 23, 2024
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.

3 participants