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

fix(l1, l2, levm) #1433

Merged
merged 40 commits into from
Dec 6, 2024
Merged
Changes from all commits
Commits
Show all changes
40 commits
Select commit Hold shift + click to select a range
55b8ae9
NOTE: REVERT LATER. Comment out CI instructions for debug purposes.
lima-limon-inc Dec 6, 2024
7480665
NOTE: REVERT LATER. Add push condition for testing purposes.
lima-limon-inc Dec 6, 2024
1689bb1
Add test channel
lima-limon-inc Dec 6, 2024
53f1562
Remove ref_name specification from key name.
lima-limon-inc Dec 6, 2024
045dcc3
Replace back tick with single quotes.
lima-limon-inc Dec 6, 2024
2cb3169
Print loc_report.json to github
lima-limon-inc Dec 6, 2024
a8cbe20
Revert "Remove ref_name specification from key name."
lima-limon-inc Dec 6, 2024
8d6abde
See previous report
lima-limon-inc Dec 6, 2024
feb75ff
Try renaming here
lima-limon-inc Dec 6, 2024
b0516fa
Mute all the channels. Sorry everyone
lima-limon-inc Dec 6, 2024
6526422
Only print old loc_report
lima-limon-inc Dec 6, 2024
991f700
Change rust code
lima-limon-inc Dec 6, 2024
c79613d
Remove loc_report move
lima-limon-inc Dec 6, 2024
e3801de
Add lines to check if the script registers new lines
lima-limon-inc Dec 6, 2024
e90ed5b
Try with unwrap_or again
lima-limon-inc Dec 6, 2024
fa75f19
Move mv instruction below run directive to make it more consistent
lima-limon-inc Dec 6, 2024
7719d12
Revert this commit to check if this was the issue
lima-limon-inc Dec 6, 2024
4924b98
Revert "Revert this commit to check if this was the issue"
lima-limon-inc Dec 6, 2024
07e01d2
Try sending result to levm slack channel
lima-limon-inc Dec 6, 2024
2a30a27
Revert "NOTE: REVERT LATER. Comment out CI instructions for debug pur…
lima-limon-inc Dec 6, 2024
900de95
Remove test expect
lima-limon-inc Dec 6, 2024
1e488ef
Revert "NOTE: REVERT LATER. Add push condition for testing purposes."
lima-limon-inc Dec 6, 2024
2720839
REmove test channel
lima-limon-inc Dec 6, 2024
351c21e
Re-enable slack channels
lima-limon-inc Dec 6, 2024
a9fbc7a
Revert "Add lines to check if the script registers new lines"
lima-limon-inc Dec 6, 2024
c271e34
Remove extra cat
lima-limon-inc Dec 6, 2024
36a3c07
Merge branch 'main' into ci-diffs
lima-limon-inc Dec 6, 2024
013963c
Remove extra and unnecessary action.
lima-limon-inc Dec 6, 2024
c40fdfa
Merge branch 'ci-diffs' of github.com:lambdaclass/ethrex into ci-diffs
lima-limon-inc Dec 6, 2024
ef7bd71
Restore run directive.
lima-limon-inc Dec 6, 2024
4f85dbb
Reapply "NOTE: REVERT LATER. Add push condition for testing purposes."
lima-limon-inc Dec 6, 2024
878e4a1
Comment out slack channels
lima-limon-inc Dec 6, 2024
764a0c3
Add test again
lima-limon-inc Dec 6, 2024
6384667
Add empty lines to multiple repos to check if the CI works
lima-limon-inc Dec 6, 2024
6df5241
Remove lines
lima-limon-inc Dec 6, 2024
596c13d
Revert "Remove extra and unnecessary action."
lima-limon-inc Dec 6, 2024
e21888c
Change script
lima-limon-inc Dec 6, 2024
1900334
Moment of truth
lima-limon-inc Dec 6, 2024
e25b8cb
Comment out test channel
lima-limon-inc Dec 6, 2024
7e10f0a
Restore the yaml.
lima-limon-inc Dec 6, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 12 additions & 1 deletion .github/workflows/daily_reports.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -124,6 +124,10 @@ jobs:
restore-keys: |
loc-report-

- name: Check previous loc report
run: |
cat loc_report.json

Comment on lines +127 to +130
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I added this directive here to read the loc_report.json that was saved in the cache.

- name: Rename cached loc_report.json to loc_report.json.old
if: steps.cache-loc-report.outputs.cache-hit == 'true'
run: mv loc_report.json loc_report.json.old
Expand All @@ -141,7 +145,7 @@ jobs:

- name: Post results in summary
run: |
echo "# `ethrex` lines of code report" >> $GITHUB_STEP_SUMMARY
echo "# 'ethrex' lines of code report" >> $GITHUB_STEP_SUMMARY
cat loc_report_github.txt >> $GITHUB_STEP_SUMMARY

- name: Post results to ethrex L1 slack channel
Expand All @@ -159,6 +163,13 @@ jobs:
url: ${{ secrets.LEVM_SLACK_WEBHOOK }}
run: sh .github/scripts/publish_loc.sh
Copy link
Contributor

Choose a reason for hiding this comment

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

why are we removing this line? isn't it needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes! That's why I was getting an error. I deleted the line by mistake.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed here: ef7bd71


# This is left commented out to ease debugging later on.
# Comment out all the other "Post to * slack channel" directives
# - name: Post results to test slack channel
# env:
# url: ${{ secrets.TEST_CHANNEL_SLACK }}
# run: sh .github/scripts/publish_loc.sh

levm-test:
name: Generate Report for LEVM EF Tests
runs-on: ubuntu-latest
Expand Down
Loading