Skip to content

Specify that the regexes are Python regexes and link to the Python docs#110

Merged
yarikoptic merged 1 commit intomasterfrom
regex-link
Jun 14, 2021
Merged

Specify that the regexes are Python regexes and link to the Python docs#110
yarikoptic merged 1 commit intomasterfrom
regex-link

Conversation

@jwodder
Copy link
Copy Markdown
Contributor

@jwodder jwodder commented Jun 14, 2021

No description provided.

@jwodder jwodder added the documentation Changes only affect the documentation label Jun 14, 2021
Copy link
Copy Markdown
Member

@yarikoptic yarikoptic left a comment

Choose a reason for hiding this comment

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

Looks LGTM, thank you @jwodder ! Sorry it took me "long" to review/approve so conflicts piled up

Comment thread README.rst
.. _DataLad: https://www.datalad.org

.. _Python regular expressions: https://docs.python.org/3/library/re.html
#regular-expression-syntax
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

note to myself: tested in github README.rst renderer for the diff -- it manages to bring the #fragment properly into the url

@jwodder
Copy link
Copy Markdown
Contributor Author

jwodder commented Jun 14, 2021

@yarikoptic Conflict resolved.

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #110 (8737b5a) into master (7ef4814) will decrease coverage by 27.72%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #110       +/-   ##
===========================================
- Coverage   72.52%   44.80%   -27.73%     
===========================================
  Files          10        9        -1     
  Lines         899      895        -4     
  Branches      134      134               
===========================================
- Hits          652      401      -251     
- Misses        193      493      +300     
+ Partials       54        1       -53     
Impacted Files Coverage Δ
src/tinuous/__main__.py 0.00% <0.00%> (-55.39%) ⬇️
src/tinuous/appveyor.py 32.14% <0.00%> (-45.24%) ⬇️
src/tinuous/travis.py 28.18% <0.00%> (-38.19%) ⬇️
src/tinuous/base.py 61.90% <0.00%> (-25.40%) ⬇️
src/tinuous/github.py 30.50% <0.00%> (-20.50%) ⬇️
src/tinuous/config.py 73.68% <0.00%> (-18.95%) ⬇️
src/tinuous/util.py 81.81% <0.00%> (-4.55%) ⬇️
src/tinuous/_version.py

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 7ef4814...8737b5a. Read the comment docs.

@yarikoptic yarikoptic merged commit ba2ae86 into master Jun 14, 2021
@yarikoptic yarikoptic deleted the regex-link branch June 14, 2021 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Changes only affect the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants