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

More coverage and typos #150

Merged
merged 12 commits into from
Aug 16, 2024
Merged

More coverage and typos #150

merged 12 commits into from
Aug 16, 2024

Conversation

mcol
Copy link
Contributor

@mcol mcol commented Aug 16, 2024

This adds tests for the following files (part of #121):

  • Second2Gray.R
  • analyse_Al2O3C_CrossTalk.R
  • analyse_SAR.CWOSL.R
  • analyse_SAR.TL.R
  • analyse_pIRIRSequence.R
  • apply_CosmicRayRemoval.R
  • calc_CosmicDoseRate.R
  • calc_FadingCorr.R
  • calc_FuchsLang2001.R
  • calc_HomogeneityTest.R
  • merge_Risoe.BINfileData.R

Note the following renames:

  • test_analyse_SARTL.R to test_analyse_SAR.TL.R
  • test_merge_RisoeBINfileData.R to test_merge_Risoe.BINfileData.R

Still a few typos found! 😄

@RLumSK RLumSK merged commit c6d0356 into R-Lum:dev_0.9.x Aug 16, 2024
10 checks passed
@RLumSK
Copy link
Member

RLumSK commented Aug 16, 2024

Thanks @mcol. For the typos one request: Please either build the documentation and then submit with the commit (RLumBuild::module_build_documentation(); stage only corresponding files) or just leave it in the R files and it gets build automatically with the next "Full rebuild".

@mcol mcol deleted the coverage_07 branch August 16, 2024 21:32
@mcol mcol restored the coverage_07 branch August 16, 2024 21:32
@mcol mcol deleted the coverage_07 branch August 16, 2024 21:32
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