Skip to content

TYP: Add type hints to the Figure.tilemap method #3095

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

Merged
merged 8 commits into from
Mar 20, 2024
Merged

Conversation

seisman
Copy link
Member

@seisman seisman commented Mar 11, 2024

@seisman seisman added skip-changelog Skip adding Pull Request to changelog needs review This PR has higher priority and needs review. typing Type hints and static type checking labels Mar 11, 2024
@seisman seisman added this to the 0.12.0 milestone Mar 11, 2024
@seisman seisman removed the needs review This PR has higher priority and needs review. label Mar 11, 2024
@seisman seisman marked this pull request as draft March 11, 2024 03:20
@seisman seisman marked this pull request as ready for review March 15, 2024 15:35
@seisman seisman added the needs review This PR has higher priority and needs review. label Mar 15, 2024
See :doc:`Contextily providers <contextily:providers_deepdive>` for a list of
tile providers. Default is ``xyzservices.providers.OpenStreetMap.HOT``, i.e.
OpenStreetMap Humanitarian web tiles.
- A web tile provider in the form of a URL. The placeholders for the XYZ in the
Copy link
Member Author

Choose a reason for hiding this comment

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

I have to use double curly brackets here {{x}}, otherwise the fmt_docstrings decorator will report KeyError exception (see https://github.com/GenericMappingTools/pygmt/actions/runs/8225955536/job/22491725235?pr=3095).

The {{x}} is correctly shown as {x} on the website.

@seisman seisman added final review call This PR requires final review and approval from a second reviewer and removed needs review This PR has higher priority and needs review. labels Mar 19, 2024
@seisman seisman merged commit 598154a into main Mar 20, 2024
@seisman seisman deleted the typing/tilemap branch March 20, 2024 02:45
@seisman seisman removed the final review call This PR requires final review and approval from a second reviewer label Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog Skip adding Pull Request to changelog typing Type hints and static type checking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant