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 all SIL-specific stuff #122

Merged
merged 3 commits into from
Aug 8, 2023
Merged

Remove all SIL-specific stuff #122

merged 3 commits into from
Aug 8, 2023

Conversation

kinke
Copy link
Contributor

@kinke kinke commented Aug 4, 2023

Moving it to SIL instead.

@codecov
Copy link

codecov bot commented Aug 4, 2023

Codecov Report

Merging #122 (6ecd824) into master (0dff1bb) will decrease coverage by 0.09%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #122      +/-   ##
==========================================
- Coverage   15.29%   15.20%   -0.09%     
==========================================
  Files          12       12              
  Lines        1419     1427       +8     
==========================================
  Hits          217      217              
- Misses       1202     1210       +8     
Files Changed Coverage Δ
source/jmap/api.d 0.00% <ø> (ø)
source/symmetry/imap/request.d 0.00% <0.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@kinke kinke force-pushed the mv_sil_stuff branch 4 times, most recently from 1fc9623 to 3de8354 Compare August 4, 2023 12:57
@kinke kinke marked this pull request as ready for review August 4, 2023 12:58
@kinke kinke marked this pull request as draft August 4, 2023 13:27
@kinke
Copy link
Contributor Author

kinke commented Aug 4, 2023

Still need to confirm @SILdoc.

@kinke
Copy link
Contributor Author

kinke commented Aug 4, 2023

No idea how it's working in detail, but @SILdoc is working as-is without depending on sil-lang. I've tested a full REPL:

> :? imap.Capability
imap.CapabilityEnumMembers
Capabilities of mail server

@kinke kinke marked this pull request as ready for review August 4, 2023 16:06
@@ -6,8 +6,9 @@ module symmetry.imap.grammar;
a PEG grammer.
+/

version (SIL) :
import pegged.grammar;
version (Have_pegged):
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This unused module was previously enabled for the SIL plugin and sil-library config. Not anymore - there's the with-grammar config now instead (build tested by CI).

@skoppe
Copy link
Contributor

skoppe commented Aug 8, 2023

No idea how it's working in detail, but @SILdoc is working as-is without depending on sil-lang.

Its an alias to some symbol in mir IIRC.

@skoppe skoppe merged commit 5c1df1c into master Aug 8, 2023
12 of 14 checks passed
@kinke kinke deleted the mv_sil_stuff branch August 8, 2023 17:02
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