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

Remove Comments & Compilation Step Only Feature #10

Merged
merged 13 commits into from
Dec 19, 2024

Conversation

H00N24
Copy link
Collaborator

@H00N24 H00N24 commented Dec 19, 2024

This PR adds:

  • java installation for type-checking support
  • remove comments from conf files so we can use jq
  • add an option for --compilation_steps_only
  • adding license

Successful execution: https://github.com/Certora/certora-run-action-test/pull/5

@H00N24 H00N24 requested a review from fnast9 December 19, 2024 14:15
Copy link

Certora Run Started (Verified Rules)

  • Group ID: 8c1412a5-ff6e-4260-b4eb-c85496c20526
Config Status Link Log File
tests/conf-verified.conf Failed (1) - tests/conf-verified.conf-134d59d0797c.log
tests/conf-verified.conf --method "counter()" Failed (1) - tests/conf-verified.conf-ed2b43bfca01.log
tests/conf-verified.conf --rule invertible Failed (1) - tests/conf-verified.conf-69fd546d063f.log
tests/conf-verified.conf --rule monotone --method "counter()" Failed (1) - tests/conf-verified.conf-c1cfe7a576f4.log

Certora Run Summary

  • Started 0 jobs
  • 4 jobs failed

Download Logs

@Certora Certora deleted a comment from github-actions bot Dec 19, 2024
@Certora Certora deleted a comment from github-actions bot Dec 19, 2024
Copy link

@fnast9 fnast9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good to me. I assume that you agreed on this specific License.

@H00N24 H00N24 merged commit 02d9937 into main Dec 19, 2024
1 check passed
@H00N24 H00N24 deleted the feat/remove-comments-and-compilation-only branch December 19, 2024 15:26
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.

2 participants