-
Notifications
You must be signed in to change notification settings - Fork 542
-
Notifications
You must be signed in to change notification settings - Fork 542
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 integration #56
Comments
Now that we have docker image on dockerhub, can we just use that instead of building it in the orb file? @sashman |
I've created a PR for the above comment: infracost/infracost-orb#2 |
I'll take this |
Working on infracost/infracost-orb#1 next |
Hi, this issue has any updates? I'm really interested to integrate infracost at circleci. |
@amioranza yep! I've simplified the CI script so it can be re-used across GitHub, GitLab and CircleCI, the refactor will be merged today/tmrw and I'll update the orb to use it next right after that :) |
Awesome!!! Thanks. |
@amioranza this is now released, please see https://github.com/infracost/infracost-orb
What's your take on infracost CI integrations storing the infracost output from the current branch as an artifact in the build, useful for future reference or is the diff comment enough? I'll close the issue but feel free to comment to let me know how it goes. |
Hi @alikhajeh1:
I think that infracost comment is enough, it we need to see the history we can go to the PR and look at the comments. |
@sashman is already working on this, https://github.com/infracost/infracost-orb
The text was updated successfully, but these errors were encountered: