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

[#207] fix wrongly detected puzzle body #208

Merged
merged 1 commit into from Jul 11, 2022
Merged

[#207] fix wrongly detected puzzle body #208

merged 1 commit into from Jul 11, 2022

Conversation

mbao01
Copy link
Contributor

@mbao01 mbao01 commented Jul 11, 2022

  • use spaces as indentation for puzzle body when no line comment prefix is found

@mbao01
Copy link
Contributor Author

mbao01 commented Jul 11, 2022

@yegor256 please review

@codecov
Copy link

codecov bot commented Jul 11, 2022

Codecov Report

Merging #208 (f09c8bc) into master (cd166ee) will increase coverage by 0.02%.
The diff coverage is 100.00%.

❗ Current head f09c8bc differs from pull request most recent head ba4b803. Consider uploading reports for the commit ba4b803 to get more accurate results

@@            Coverage Diff             @@
##           master     #208      +/-   ##
==========================================
+ Coverage   95.75%   95.77%   +0.02%     
==========================================
  Files          10       10              
  Lines         353      355       +2     
==========================================
+ Hits          338      340       +2     
  Misses         15       15              
Impacted Files Coverage Δ
lib/pdd/source.rb 99.16% <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 cd166ee...ba4b803. Read the comment docs.

@yegor256
Copy link
Member

@rultor merge

@rultor
Copy link
Collaborator

rultor commented Jul 11, 2022

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit ba4b803 into cqfn:master Jul 11, 2022
@rultor
Copy link
Collaborator

rultor commented Jul 11, 2022

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 1min)

@yegor256
Copy link
Member

@rultor release, tag is 0.21.1

@rultor
Copy link
Collaborator

rultor commented Jul 11, 2022

@rultor release, tag is 0.21.1

@yegor256 OK, I will release it now. Please check the progress here

@rultor
Copy link
Collaborator

rultor commented Jul 11, 2022

@rultor release, tag is 0.21.1

@yegor256 Done! FYI, the full log is here (took me 2min)

@yegor256
Copy link
Member

@mbao01 thanks!

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.

None yet

3 participants