-
Notifications
You must be signed in to change notification settings - Fork 273
Remove JSIL front-end #8158
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 JSIL front-end #8158
Conversation
We do not have any tests and JSIL is not being developed any further at https://vtss.doc.ic.ac.uk/research/javascript.html.
46d52e1
to
67be193
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #8158 +/- ##
===========================================
+ Coverage 79.09% 79.69% +0.59%
===========================================
Files 1695 1680 -15
Lines 196589 195104 -1485
===========================================
- Hits 155493 155479 -14
+ Misses 41096 39625 -1471 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approve in the sense that I believe this is technically correct.
I have emailed Philippa to see if she has any thoughts on this but given POPL starts in a few days I think it might take a little while to hear back.
I firmly believe in version control and am trying to keep commits clean. We should be able to revert quite easily if the need arises. |
We do not have any tests and JSIL is not being developed any further at https://vtss.doc.ic.ac.uk/research/javascript.html.