Skip to content

Conversation

@MichaelChirico
Copy link
Collaborator

Part of #884

A few more odds & ends to tie up, filing early for any initial comments

@codecov-commenter
Copy link

codecov-commenter commented Oct 7, 2022

Codecov Report

Merging #1635 (9e4de5a) into main (16e8576) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1635      +/-   ##
==========================================
+ Coverage   98.31%   98.33%   +0.01%     
==========================================
  Files         100      101       +1     
  Lines        4397     4443      +46     
==========================================
+ Hits         4323     4369      +46     
  Misses         74       74              
Impacted Files Coverage Δ
R/is_numeric_linter.R 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@MichaelChirico MichaelChirico changed the title WIP: New is_numeric_linter New is_numeric_linter Oct 7, 2022
@IndrajeetPatil
Copy link
Collaborator

Resolved merge conflict.

@MichaelChirico
Copy link
Collaborator Author

Resolved merge conflict.

thanks... FWIW I have a setting in my local got config that always resolves merges on our NEWS file correctly. shame I couldn't figure out how to make it work on the site last I checked

@MichaelChirico
Copy link
Collaborator Author

# Merge made by the 'ort' strategy.

FYI this is my merge setting locally

Copy link
Collaborator

@IndrajeetPatil IndrajeetPatil left a comment

Choose a reason for hiding this comment

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

Thanks!

@IndrajeetPatil IndrajeetPatil merged commit c9731e3 into main Oct 9, 2022
@IndrajeetPatil IndrajeetPatil deleted the is-numeric branch October 9, 2022 06:12
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