Skip to content

Comments

Fix #397 - extern(C++, ...) only supports identifiers#398

Merged
dlang-bot merged 1 commit intodlang-community:masterfrom
MoonlightSentinel:string-ns
May 2, 2020
Merged

Fix #397 - extern(C++, ...) only supports identifiers#398
dlang-bot merged 1 commit intodlang-community:masterfrom
MoonlightSentinel:string-ns

Conversation

@MoonlightSentinel
Copy link
Contributor

Implemented according to the grammar defined in the pending spec PR

@codecov
Copy link

codecov bot commented Apr 30, 2020

Codecov Report

Merging #398 into master will increase coverage by 0.14%.
The diff coverage is 97.22%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #398      +/-   ##
==========================================
+ Coverage   76.12%   76.26%   +0.14%     
==========================================
  Files           9        9              
  Lines        7132     7168      +36     
==========================================
+ Hits         5429     5467      +38     
+ Misses       1703     1701       -2     
Impacted Files Coverage Δ
src/dparse/ast.d 55.88% <97.05%> (+3.84%) ⬆️
src/dparse/parser.d 90.94% <100.00%> (+<0.01%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 773ef19...914ff96. Read the comment docs.

@MoonlightSentinel MoonlightSentinel force-pushed the string-ns branch 8 times, most recently from abb90e2 to 28f4d4e Compare May 1, 2020 09:01
Implemented according to the grammar defined in the pending spec PR 2716
Copy link
Member

@Geod24 Geod24 left a comment

Choose a reason for hiding this comment

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

LGTM, but I have no power here.

@dlang-bot dlang-bot merged commit 799a96b into dlang-community:master May 2, 2020
@MoonlightSentinel MoonlightSentinel deleted the string-ns branch May 4, 2020 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants