Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

make executables bold #405

Merged
merged 1 commit into from
Oct 23, 2020
Merged

Conversation

t-mangoe
Copy link
Contributor

Description

I added feature that show executables files using boldface.

  • Relevant Issues : Make executables bold #217
  • Relevant PRs : (none)
  • Type of change :
    • New feature
    • Bug fix for existing feature
    • Code quality improvement
    • Addition or Improvement of tests
    • Addition or Improvement of documentation

@codecov-io
Copy link

codecov-io commented Oct 17, 2020

Codecov Report

Merging #405 into master will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #405      +/-   ##
==========================================
+ Coverage   88.99%   89.01%   +0.02%     
==========================================
  Files           8        8              
  Lines         518      519       +1     
==========================================
+ Hits          461      462       +1     
  Misses         57       57              
Impacted Files Coverage Δ
lib/colorls/core.rb 90.30% <100.00%> (+0.04%) ⬆️
lib/colorls/fileinfo.rb 79.48% <100.00%> (ø)

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 6f9bc84...9992218. Read the comment docs.

@avdv avdv merged commit 041963a into athityakumar:master Oct 23, 2020
@avdv
Copy link
Collaborator

avdv commented Oct 23, 2020

Sorry for the late response, but I am in vacation currently... and I just wanted to be sure that bright is the thing we want, and it is indeed an alias for bold.

Thank you!

@avdv avdv mentioned this pull request Oct 23, 2020
5 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants