Improve accessibility of links & clarify technical language & correct spelling

[RayStick]

Closes #185

@Lextuga007 this PR is responding to 3 of your review comments, are you okay to review it? No problem if not, I can get another collaborator to do so. I just wanted to check I had responded to your comments appropriately, specifically regarding accessibility of links.

Checklist for the author of this PR:

• [if package files were edited] I have run these checks locally:
  • devtools::document() to generates the .Rd files from any updated roxygen comments.
  • codemetar::write_codemeta() to ensures the metadata file is up to date.
  • styler::style_pkg() to ensure consistent code styling that match the guidelines.
  • devtools::check() for a comprehensive package check. I have resolved any warnings or errors, or written them here in the PR, for discussion.
• The code base and the documentation files match (they both reflect any recent changes).
• The title of this PR is clear and self-explantory.
• I have added any appropriate labels to this PR.
• This PR is now ready for review (and I have removed the draft PR status).

[Lextuga007]

Should the name of the file also be changed?

[RayStick]

the function 'consensus_on_mismatch' is within
https://github.com/aim-rsf/mapmetadata/blob/R2-2_R2-3_R2-5/R/compare.R
So we should be sorted on all the renaming :)
Let me know if I am missing something!

[Lextuga007]

I could have been clearer - apologies! The file itself is called test-consensus_on_mistmatch.R and now that the function has been changed the file name should change too I guess?

[RayStick]

Yep I follow =D
This filename has been changed in the branch: https://github.com/aim-rsf/mapmetadata/blob/R2-2_R2-3_R2-5/tests/testthat/test-consensus_on_mismatch.R
Is that what you mean? Apologies if I am missing something obvious

[RayStick]

As in it used to be

tests/testthat/test-concensus_on_mismatch.R

now it is

tests/testthat/test-consensus_on_mismatch.R

[codecov]

Codecov Report

Attention: Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
R/metadata_map.R	66.66%	1 Missing ⚠️

Files with missing lines	Coverage Δ
R/compare.R	100.00% <100.00%> (ø)
R/map_compare.R	100.00% <100.00%> (ø)
R/metadata_map.R	73.24% <66.66%> (ø)

[RayStick]

@Lextuga007 thanks for your comments, I am going to merge this branch into main if that's okay
(Don't worry about the code coverage check failing, I will resolve that later)